kitchen-rackspace 0.11.0 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b38ffcc8eb26f872550c5ec737ed5066857fb6c4
-  data.tar.gz: 68bdd5ae2e14296e510c3459441e1ad898c1ec38
+  metadata.gz: 02c665cc850869f22e3b31463bc869f67d11b8e7
+  data.tar.gz: 8ae8afc951aa645aedb9e30edc1a198b80b9991d
 SHA512:
-  metadata.gz: bf5a93011bdb76cf76162077670dc66c6db8a0dd41c91a00d8bcc394c0b99ae8fd5a079b5652af8c39d7a1fe73ceadf337c9864c05fb06d8829bbe4b92e74bf9
-  data.tar.gz: 1402487a8c35aacbd2189b1e020fe5c9c23bc3fc549137e80267313eef581e03f354fa0fdc83bffdf07647a966ba9ddb1fa5e32e5e736a2e6bb744fe3f140f4c
+  metadata.gz: 45dcd888d5027beb598964b12c84abd12ef48c48d7010f3040b44556fea280307e1661276d2df02ecb1de1da154ad9ffefea876b2a9931332d05c2521792554c
+  data.tar.gz: fbade3eb3f36f632c1960ccc52fc20c4fc7987ce1c74cb418b6f5ba92ecb54f72487632c379e0b5a5ea20a65779ca957092c71126b6710f12c3812ca615444d0

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+# 0.12.0 / 2014-09-10
+
+### New Features
+
+* PR [#41][] - Support optionally using ServiceNet for SSH access, via
+[@steve-jansen][]
+
 # 0.11.0 / 2014-09-04
 
 ### Improvements
@@ -97,6 +104,7 @@ boot times, via [@coderanger][]
 
 * Initial release! Woo!
 
+[#41]: https://github.com/test-kitchen/kitchen-rackspace/pull/41
 [#40]: https://github.com/test-kitchen/kitchen-rackspace/pull/40
 [#39]: https://github.com/test-kitchen/kitchen-rackspace/pull/39
 [#38]: https://github.com/test-kitchen/kitchen-rackspace/pull/38
@@ -117,6 +125,7 @@ boot times, via [@coderanger][]
 [#8]: https://github.com/test-kitchen/kitchen-rackspace/pull/8
 [#7]: https://github.com/test-kitchen/kitchen-rackspace/pull/7
 
+[@steve-jansen]: https://github.com/steve-jansen
 [@hhoover]: https://github.com/hhoover
 [@kanerogers]: https://github.com/kanerogers
 [@martinb3]: https://github.com/martinb3

data/README.md

@@ -42,7 +42,7 @@ for your specified platform. Additional, optional overrides can be provided:
 
     image_id: [SERVER IMAGE ID]
     flavor_id: [SERVER FLAVOR ID]
-    server_name: [A UNIQUE SERVER NAME]
+    server_name: [A FRIENDLY SERVER NAME]
     public_key_path: [PATH TO YOUR PUBLIC SSH KEY]
     rackspace_region: [A VALID RACKSPACE DC/REGION]
     wait_for: [NUM OF SECONDS TO WAIT BEFORE TIMING OUT, DEFAULT 600]
@@ -50,6 +50,7 @@ for your specified platform. Additional, optional overrides can be provided:
     no_ssh_tcp_check_sleep: [NUM OF SECONDS TO SLEEP IF no_ssh_tcp_check IS SET]
     networks: [LIST OF RACKSPACE NETWORK UUIDS, DEFAULT PUBLICNET AND SERVICE NET]
     rackconnect_wait: ['true' IF USING RACKCONNECT TO WAIT FOR IT TO COMPLETE]
+    servicenet: ['true' IF USING THE SERVICENET IP ADDRESS TO CONNECT]
 
 You also have the option of providing some configs via environment variables:
 
@@ -66,7 +67,9 @@ Some configs are also derived based on your .ssh directory, specifically the
 ## Contributing
 
 1. Fork it
-2. Create your feature branch (`git checkout -b my-new-feature`)
-3. Commit your changes (`git commit -am 'Add some feature'`)
-4. Push to the branch (`git push origin my-new-feature`)
-5. Create new Pull Request
+2. `bundle install`
+3. Create your feature branch (`git checkout -b my-new-feature`)
+4. `bundle exec rake` must pass
+5. Commit your changes (`git commit -am 'Add some feature'`)
+6. Push to the branch (`git push origin my-new-feature`)
+7. Create new Pull Request

data/lib/kitchen/driver/rackspace.rb

@@ -37,6 +37,7 @@ module Kitchen
       default_config :no_ssh_tcp_check, false
       default_config :no_ssh_tcp_check_sleep, 120
       default_config :rackconnect_wait, false
+      default_config :servicenet, false
       default_config(:image_id) { |driver| driver.default_image }
       default_config(:server_name) { |driver| driver.default_name }
       default_config :networks, nil
@@ -75,7 +76,7 @@ module Kitchen
         server.wait_for { ready? }
         puts '(server ready)'
         rackconnect_check(server) if config[:rackconnect_wait]
-        state[:hostname] = server.public_ip_address
+        state[:hostname] = hostname(server)
         tcp_check(state)
       rescue Fog::Errors::Error, Excon::Errors::Error => ex
         raise ActionFailed, ex.message
@@ -156,6 +157,14 @@ module Kitchen
         server.update # refresh accessIPv4 with new IP
       end
 
+      def hostname(server)
+        if config[:servicenet] == false
+          server.public_ip_address
+        else
+          server.private_ip_address
+        end
+      end
+
       def networks
         base_nets = %w(
           00000000-0000-0000-0000-000000000000

data/lib/kitchen/driver/rackspace_version.rb

@@ -21,6 +21,6 @@ module Kitchen
   #
   # @author Jonathan Hartman <j@p4nt5.com>
   module Driver
-    RACKSPACE_VERSION = '0.11.0'
+    RACKSPACE_VERSION = '0.12.0'
   end
 end

data/spec/kitchen/driver/rackspace_spec.rb

@@ -113,6 +113,10 @@ describe Kitchen::Driver::Rackspace do
       it 'defaults to not waiting for rackconnect' do
         expect(driver[:rackconnect_wait]).to eq(false)
       end
+
+      it 'defaults to the public ip address' do
+        expect(driver[:servicenet]).to eq(false)
+      end
     end
 
     platforms = {
@@ -143,7 +147,8 @@ describe Kitchen::Driver::Rackspace do
         server_name: 'puppy',
         rackspace_region: 'ord',
         wait_for: 1200,
-        rackconnect_wait: true
+        rackconnect_wait: true,
+        use_private_ip_address: true
       }
 
       let(:config) { config }
@@ -223,7 +228,108 @@ describe Kitchen::Driver::Rackspace do
         driver.create(state)
       end
     end
+  end
+
+  describe '#create and rackconnect_wait' do
+    let(:server) do
+      double(id: 'test123',
+             wait_for: true,
+             public_ip_address: '1.2.3.4',
+             private_ip_address: '10.9.8.7',
+             update: nil)
+    end
+    let(:driver) do
+      d = Kitchen::Driver::Rackspace.new(config)
+      d.instance = instance
+      allow(d).to receive(:default_name).and_return('a_monkey!')
+      allow(d).to receive(:create_server).and_return(server)
+      allow(d).to receive(:tcp_check).and_return(true)
+      d
+    end
 
+    context 'username and API key only provided' do
+      let(:config) do
+        {
+          rackspace_username: 'hello',
+          rackspace_api_key: 'world',
+          wait_for: 1200,
+          rackconnect_wait: true
+        }
+      end
+
+      it 'generates a server name in the absence of one' do
+        driver.create(state)
+        expect(driver[:server_name]).to eq('a_monkey!')
+      end
+
+      it 'gets a proper server ID' do
+        driver.create(state)
+        expect(state[:server_id]).to eq('test123')
+      end
+
+      it 'gets a proper hostname (IP)' do
+        driver.create(state)
+        expect(state[:hostname]).to eq('1.2.3.4')
+      end
+
+      it 'calls tcp_check' do
+        expect(driver).to receive(:tcp_check)
+        driver.create(state)
+      end
+
+      it 'calls rackconnect_check ' do
+        expect(driver).to receive(:rackconnect_check)
+        driver.create(state)
+      end
+
+      it 'rackconnect_check waits for rackconnect_automation' do
+        expect(server).to receive(:wait_for)
+        driver.send(:rackconnect_check, server)
+      end
+    end
+  end
+
+  describe '#create and use_private_ip_address' do
+    let(:server) do
+      double(id: 'test123',
+             wait_for: true,
+             public_ip_address: '1.2.3.4',
+             private_ip_address: '10.9.8.7')
+    end
+    let(:driver) do
+      d = Kitchen::Driver::Rackspace.new(config)
+      d.instance = instance
+      allow(d).to receive(:default_name).and_return('a_monkey!')
+      allow(d).to receive(:create_server).and_return(server)
+      allow(d).to receive(:tcp_check).and_return(true)
+      d
+    end
+
+    context 'username and API key only provided' do
+      let(:config) do
+        {
+          rackspace_username: 'hello',
+          rackspace_api_key: 'world',
+          wait_for: 1200,
+          servicenet: true
+        }
+      end
+
+      it 'generates a server name in the absence of one' do
+        driver.create(state)
+        expect(driver[:server_name]).to eq('a_monkey!')
+      end
+
+      it 'gets a proper server ID' do
+        driver.create(state)
+        expect(state[:server_id]).to eq('test123')
+      end
+
+      it 'gets a private ip as the hostname' do
+        driver.create(state)
+        expect(state[:hostname]).to eq('10.9.8.7')
+      end
+    end
   end
 
   describe '#destroy' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-rackspace
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.12.0
 platform: ruby
 authors:
 - Jonathan Hartman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-04 00:00:00.000000000 Z
+date: 2014-09-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: test-kitchen