kitchen-ec2 0.9.3 → 0.9.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 21b2743d4aad92b8fb4f5d02d3619c0b1b83c6eb
-  data.tar.gz: 6dc57776d894c34fad0da0701e4c6d4a59076698
+  metadata.gz: ce1e378058d18afcc07e2f59a986588f9e05887f
+  data.tar.gz: 2e902c288551eab23b6f0ed314dd7616277a13de
 SHA512:
-  metadata.gz: 3358dc9d1df3053b5f6efaf4f4ec2673f46eb3b81a1b81d080a22f92203bf851abe1133429a16dd5d78adf0ca1cf5d62298c632b011b58d1ce3d9651e4dcc950
-  data.tar.gz: 430c0e2766480db4d1053afa73f2b609d9379e283d139fe2e6a36cfbf9f21e386ab0853ec26b417ec5fcfda6e640775b0b31a5af5c80c6f58961d4457498bd62
+  metadata.gz: ef7beeddeeca20586ef7800e42c841b530f04fc2bc16098e79478e37654770c1bf163ea9a9e3707827d3f060eda2b71886eb50d7b3123740932d060dcac0297f
+  data.tar.gz: 3bd25b6b3654267dfe2262a796659e647749f57fd1095551e414ded3c5fc5f327965eb0f2b719369e290c94536ed5ff6fd2aef265340e84af9927c44495f8253

data/.rubocop.yml

@@ -13,3 +13,6 @@ Metrics/CyclomaticComplexity:
 
 Metrics/PerceivedComplexity:
   Max: 30
+
+Metrics/AbcSize:
+  Max: 60

data/CHANGELOG.md

@@ -1,3 +1,13 @@
+## 0.9.4 / 2015-06-03
+
+### Bug Fixes
+
+* Pull Request [#142][]: Fixing NoMethodError, providing logger to instance_generator
+
+### New Features
+
+### Improvements
+
 ## 0.9.3 / 2015-05-29
 
 ### Bug Fixes
@@ -154,6 +164,7 @@
 [#128]: https://github.com/test-kitchen/kitchen-ec2/issues/128
 [#131]: https://github.com/test-kitchen/kitchen-ec2/issues/131
 [#140]: https://github.com/test-kitchen/kitchen-ec2/issues/140
+[#142]: https://github.com/test-kitchen/kitchen-ec2/issues/142
 [@Atalanta]: https://github.com/Atalanta
 [@Igorshp]: https://github.com/Igorshp
 [@JamesAwesome]: https://github.com/JamesAwesome

data/README.md

@@ -425,7 +425,7 @@ Apache 2.0 (see [LICENSE][license])
 [issues]:           https://github.com/test-kitchen/kitchen-ec2/issues
 [license]:          https://github.com/test-kitchen/kitchen-ec2/blob/master/LICENSE
 [repo]:             https://github.com/test-kitchen/kitchen-ec2
-[driver_usage]:     http://docs.kitchen-ci.org/drivers/usage
+[driver_usage]:     https://github.com/test-kitchen/kitchen-ec2
 [chef_omnibus_dl]:  http://www.getchef.com/chef/install/
 
 [amis_json]:        https://github.com/test-kitchen/kitchen-ec2/blob/master/data/amis.json

data/lib/kitchen/driver/aws/instance_generator.rb

@@ -30,13 +30,12 @@ module Kitchen
       # @author Tyler Ball <tball@chef.io>
       class InstanceGenerator
 
-        include Logging
+        attr_reader :config, :ec2, :logger
 
-        attr_reader :config, :ec2
-
-        def initialize(config, ec2)
+        def initialize(config, ec2, logger)
           @config = config
           @ec2 = ec2
+          @logger = logger
         end
 
         # Transform the provided config into the hash to send to AWS.  Some fields
@@ -125,6 +124,7 @@ module Kitchen
         # If the provided bdms match the root device in the AMI, emit log that
         # states this
         def debug_if_root_device(bdms)
+          return if bdms.nil? || bdms.empty?
           image_id = config[:image_id]
           image = ec2.resource.image(image_id)
           begin
@@ -136,7 +136,7 @@ module Kitchen
           end
           bdms.find { |bdm|
             if bdm[:device_name] == root_device_name
-              info("Overriding root device [#{root_device_name}] from image [#{image_id}]")
+              logger.info("Overriding root device [#{root_device_name}] from image [#{image_id}]")
             end
           }
         end

data/lib/kitchen/driver/ec2.rb

@@ -159,6 +159,8 @@ module Kitchen
 
         if config[:availability_zone].nil?
           config[:availability_zone] = config[:region] + "b"
+        elsif config[:availability_zone] =~ /^[a-z]$/
+          config[:availability_zone] = config[:region] + config[:availability_zone]
         end
         # TODO: when we get rid of flavor_id, move this to a default
         if config[:instance_type].nil?
@@ -196,27 +198,7 @@ module Kitchen
 
         state[:server_id] = server.id
         info("EC2 instance <#{state[:server_id]}> created.")
-        wait_log = proc do |attempts|
-          c = attempts * config[:retryable_sleep]
-          t = config[:retryable_tries] * config[:retryable_sleep]
-          info "Waited #{c}/#{t}s for instance <#{state[:server_id]}> to become ready."
-        end
-        begin
-          server = server.wait_until(
-            :max_attempts => config[:retryable_tries],
-            :delay => config[:retryable_sleep],
-            :before_attempt => wait_log
-          ) do |s|
-            hostname = hostname(s, config[:interface])
-            # Euca instances often report ready before they have an IP
-            s.exists? && s.state.name == "running" && !hostname.nil? && hostname != "0.0.0.0"
-          end
-        rescue ::Aws::Waiters::Errors::WaiterFailed
-          error("Ran out of time waiting for the server with id [#{state[:server_id]}]" \
-            " to become ready, attempting to destroy it")
-          destroy(state)
-          raise
-        end
+        wait_until_ready(server, state)
 
         info("EC2 instance <#{state[:server_id]}> ready.")
         state[:hostname] = hostname(server)
@@ -262,7 +244,7 @@ module Kitchen
       end
 
       def instance_generator
-        @instance_generator ||= Aws::InstanceGenerator.new(config, ec2)
+        @instance_generator ||= Aws::InstanceGenerator.new(config, ec2, instance.logger)
       end
 
       # This copies transport config from the current config object into the
@@ -334,6 +316,30 @@ module Kitchen
         server.create_tags(:tags => tags)
       end
 
+      def wait_until_ready(server, state)
+        wait_log = proc do |attempts|
+          c = attempts * config[:retryable_sleep]
+          t = config[:retryable_tries] * config[:retryable_sleep]
+          info "Waited #{c}/#{t}s for instance <#{state[:server_id]}> to become ready."
+        end
+        begin
+          server.wait_until(
+            :max_attempts => config[:retryable_tries],
+            :delay => config[:retryable_sleep],
+            :before_attempt => wait_log
+          ) do |s|
+            hostname = hostname(s, config[:interface])
+            # Euca instances often report ready before they have an IP
+            s.exists? && s.state.name == "running" && !hostname.nil? && hostname != "0.0.0.0"
+          end
+        rescue ::Aws::Waiters::Errors::WaiterFailed
+          error("Ran out of time waiting for the server with id [#{state[:server_id]}]" \
+            " to become ready, attempting to destroy it")
+          destroy(state)
+          raise
+        end
+      end
+
       def amis
         @amis ||= begin
           json_file = File.join(File.dirname(__FILE__),

data/lib/kitchen/driver/ec2_version.rb

@@ -21,6 +21,6 @@ module Kitchen
   module Driver
 
     # Version string for EC2 Test Kitchen driver
-    EC2_VERSION = "0.9.3"
+    EC2_VERSION = "0.9.4"
   end
 end

data/spec/kitchen/driver/ec2/instance_generator_spec.rb

@@ -24,12 +24,10 @@ require "base64"
 describe Kitchen::Driver::Aws::InstanceGenerator do
 
   let(:config) { Hash.new }
-
   let(:resource) { instance_double(Aws::EC2::Resource) }
-
   let(:ec2) { instance_double(Kitchen::Driver::Aws::Client, :resource => resource) }
-
-  let(:generator) { Kitchen::Driver::Aws::InstanceGenerator.new(config, ec2) }
+  let(:logger)        { instance_double(Logger) }
+  let(:generator) { Kitchen::Driver::Aws::InstanceGenerator.new(config, ec2, logger) }
 
   describe "#debug_if_root_device" do
     let(:image_id) { "ami-123456" }
@@ -37,20 +35,28 @@ describe Kitchen::Driver::Aws::InstanceGenerator do
     let(:image) { double("image") }
 
     before do
-      expect(resource).to receive(:image).with(image_id).and_return(image)
+      allow(resource).to receive(:image).with(image_id).and_return(image)
+    end
+
+    it "returns nil when provided nil" do
+      expect(generator.debug_if_root_device(nil)).to eq(nil)
+    end
+
+    it "returns nil when provided empty" do
+      expect(generator.debug_if_root_device([])).to eq(nil)
     end
 
     it "returns nil when the image cannot be found" do
       expect(image).to receive(:root_device_name).and_raise(
         ::Aws::EC2::Errors::InvalidAMIIDNotFound.new({}, "")
       )
-      expect(generator).to_not receive(:info)
-      expect(generator.debug_if_root_device({})).to eq(nil)
+      expect(logger).to_not receive(:info)
+      expect(generator.debug_if_root_device([{ :device_name => "name" }])).to eq(nil)
     end
 
     it "logs an info message when the device mappings are overriding the root device" do
       expect(image).to receive(:root_device_name).and_return("name")
-      expect(generator).to receive(:info)
+      expect(logger).to receive(:info)
       expect(generator.debug_if_root_device([{ :device_name => "name" }])).to eq(nil)
     end
   end

data/spec/kitchen/driver/ec2_spec.rb

@@ -88,6 +88,12 @@ describe Kitchen::Driver::Ec2 do
       driver.finalize_config!(instance)
       expect(config[:availability_zone]).to eq("us-east-1b")
     end
+
+    it "expands the availability zone if provided as a letter" do
+      config[:availability_zone] = "d"
+      driver.finalize_config!(instance)
+      expect(config[:availability_zone]).to eq("us-east-1d")
+    end
   end
 
   describe "#hostname" do
@@ -171,6 +177,10 @@ describe Kitchen::Driver::Ec2 do
   end
 
   describe "#submit_server" do
+    before do
+      expect(driver).to receive(:instance).at_least(:once).and_return(instance)
+    end
+
     it "submits the server request" do
       expect(generator).to receive(:ec2_instance_data).and_return({})
       expect(client).to receive(:create_instance).with(:min_count => 1, :max_count => 1)
@@ -184,6 +194,10 @@ describe Kitchen::Driver::Ec2 do
       { :spot_instance_requests => [{ :spot_instance_request_id => "id" }] }
     }
 
+    before do
+      expect(driver).to receive(:instance).at_least(:once).and_return(instance)
+    end
+
     it "submits the server request" do
       expect(generator).to receive(:ec2_instance_data).and_return({})
       expect(actual_client).to receive(:request_spot_instances).with(

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-ec2
 version: !ruby/object:Gem::Version
-  version: 0.9.3
+  version: 0.9.4
 platform: ruby
 authors:
 - Fletcher Nichol
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-29 00:00:00.000000000 Z
+date: 2015-06-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: test-kitchen