knife-cloudformation 0.1.20 → 0.1.22

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## v0.1.22
+* Prevent full stack list loading in knife commands
+* Default logger to INFO level and allow DEBUG level via `ENV['DEBUG']`
+* Fix assumption of type when accessing cached data (cannot assume availability)
+
 ## v0.1.20
 * Update some caching behavior
 * Add more logging especially around remote calls

data/lib/knife-cloudformation/aws_commons/stack.rb

@@ -314,15 +314,19 @@ module KnifeCloudformation
           100
         else
           all_events = events(:all)
-          total_expected = template['Resources'].size
-          action = performing
-          start = event_start_index(all_events, "#{action}_in_progress".to_sym)
-          finished = all_events.find_all do |e|
-            e['ResourceStatus'] == "#{action}_complete".upcase ||
-            e['ResourceStatus'] == "#{action}_failed".upcase
-          end.size
-          calculated = ((finished / total_expected.to_f) * 100).to_i
-          calculated < min ? min : calculated
+          if(all_events)
+            total_expected = template['Resources'].size
+            action = performing
+            start = event_start_index(all_events, "#{action}_in_progress".to_sym)
+            finished = all_events.find_all do |e|
+              e['ResourceStatus'] == "#{action}_complete".upcase ||
+              e['ResourceStatus'] == "#{action}_failed".upcase
+            end.size
+            calculated = ((finished / total_expected.to_f) * 100).to_i
+            calculated < min ? min : calculated
+          else
+            100 # Assume deletion and no events == complete
+          end
         end
       end
 

data/lib/knife-cloudformation/aws_commons.rb

@@ -18,6 +18,7 @@ module KnifeCloudformation
         unless(@logger)
           require 'logger'
           @logger = Logger.new($stdout)
+          @logger.level = ENV['DEBUG'] ? Logger::DEBUG : Logger::INFO
         end
         @logger
       end
@@ -133,8 +134,12 @@ module KnifeCloudformation
           end
         end
       end
-      @memo[:stacks].value.find_all do |s|
-        status.include?(s['StackStatus'])
+      if(@memo[:stacks].value)
+        @memo[:stacks].value.find_all do |s|
+          status.include?(s['StackStatus'])
+        end
+      else
+        []
       end
     end
 
@@ -200,7 +205,7 @@ module KnifeCloudformation
     def stack(*names)
       direct_load = names.delete(:ignore_seeds)
       result = names.map do |name|
-        [name, name.start_with?('arn:') ? name : id_from_stack_name(name)]
+        [name, name.start_with?('arn:') || direct_load ? name : id_from_stack_name(name)]
       end.map do |name, s_id|
         unless(@local[:stacks][s_id])
           unless(direct_load)

data/lib/knife-cloudformation/version.rb

@@ -2,5 +2,5 @@ module KnifeCloudformation
   class Version < Gem::Version
   end
 
-  VERSION = Version.new('0.1.20')
+  VERSION = Version.new('0.1.22')
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: knife-cloudformation
 version: !ruby/object:Gem::Version
-  version: 0.1.20
+  version: 0.1.22
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-10 00:00:00.000000000 Z
+date: 2013-12-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chef
@@ -150,9 +150,9 @@ files:
 - Gemfile
 - README.md
 - knife-cloudformation.gemspec
+- knife-cloudformation-0.1.20.gem
 - CHANGELOG.md
 - Gemfile.lock
-- knife-cloudformation-0.1.18.gem
 homepage: http://github.com/heavywater/knife-cloudformation
 licenses: []
 post_install_message: