sfn 3.0.30 → 3.0.32

data/lib/sfn/command/events.rb

@@ -1,4 +1,4 @@
-require 'sfn'
+require "sfn"
 
 module Sfn
   class Command
@@ -25,7 +25,7 @@ module Sfn
                   allowed_attributes.each do |attr|
                     width_val = events.map { |e| e[attr].to_s.length }.push(attr.length).max + 2
                     width_val = width_val > 70 ? 70 : width_val < 20 ? 20 : width_val
-                    column attr.split('_').map(&:capitalize).join(' '), :width => width_val
+                    column attr.split("_").map(&:capitalize).join(" "), :width => width_val
                   end
                 end
                 events.each do |event|
@@ -70,7 +70,7 @@ module Sfn
               i_events = i_stack.events.all
             end
           rescue => e
-            if e.class.to_s.start_with?('Errno')
+            if e.class.to_s.start_with?("Errno")
               ui.warn "Connection error encountered: #{e.message} (retrying)"
               ui.debug "#{e.class}: #{e}\n#{e.backtrace.join("\n")}"
             else
@@ -93,8 +93,8 @@ module Sfn
           unless config[:all_events]
             start_index = stack_events.rindex do |item|
               item[:stack_name] == stack.name &&
-                item[:resource_state].to_s.end_with?('in_progress') &&
-                item[:resource_status_reason].to_s.downcase.include?('user init')
+                item[:resource_state].to_s.end_with?("in_progress") &&
+                item[:resource_status_reason].to_s.downcase.include?("user init")
             end
             if start_index
               stack_events.slice!(0, start_index)
@@ -121,7 +121,7 @@ module Sfn
       def allowed_attributes
         result = super
         unless @stacks.size > 1
-          result.delete('stack_name')
+          result.delete("stack_name")
         end
         result
       end

data/lib/sfn/command/export.rb

@@ -1,4 +1,4 @@
-require 'sfn'
+require "sfn"
 
 module Sfn
   class Command
@@ -9,10 +9,10 @@ module Sfn
 
       # Run export action
       def execute!
-        raise NotImplementedError.new 'Implementation updates required'
+        raise NotImplementedError.new "Implementation updates required"
         stack_name = name_args.first
-        ui.info "#{ui.color('Stack Export:', :bold)} #{stack_name}"
-        ui.confirm 'Perform export'
+        ui.info "#{ui.color("Stack Export:", :bold)} #{stack_name}"
+        ui.confirm "Perform export"
         stack = provider.stacks.get(stack_name)
         if stack
           export_options = Smash.new.tap do |opts|
@@ -27,10 +27,10 @@ module Sfn
             write_to_bucket(result, stack),
           ].compact
           if outputs.empty?
-            ui.warn 'No persistent output location defined. Printing export:'
+            ui.warn "No persistent output location defined. Printing export:"
             ui.info _format_json(result)
           end
-          ui.info "#{ui.color('Stack export', :bold)} (#{name_args.first}): #{ui.color('complete', :green)}"
+          ui.info "#{ui.color("Stack export", :bold)} (#{name_args.first}): #{ui.color("complete", :green)}"
           unless outputs.empty?
             outputs.each do |output|
               ui.info ui.color("  -> #{output}", :blue)
@@ -71,10 +71,10 @@ module Sfn
             config[:path],
             export_file_name(stack)
           )
-          _, bucket, path = full_path.split('/', 3)
+          _, bucket, path = full_path.split("/", 3)
           directory = provider.service_for(:storage,
                                            :provider => :local,
-                                           :local_root => '/').directories.get(bucket)
+                                           :local_root => "/").directories.get(bucket)
           file_store(payload, path, directory)
         end
       end

data/lib/sfn/command/graph.rb

@@ -1,11 +1,11 @@
-require 'sfn'
-require 'graph'
+require "sfn"
+require "graph"
 
 module Sfn
   class Command
     # Graph command
     class Graph < Command
-      autoload :Provider, 'sfn/command/graph/provider'
+      autoload :Provider, "sfn/command/graph/provider"
 
       include Sfn::CommandModule::Base
       include Sfn::CommandModule::Template
@@ -13,8 +13,8 @@ module Sfn
 
       # Valid graph styles
       GRAPH_STYLES = [
-        'creation',
-        'dependency',
+        "creation",
+        "dependency",
       ]
 
       # Generate graph
@@ -33,23 +33,23 @@ module Sfn
             ui.puts "  -> path: #{config[:file]}"
           end
           template_dump = file.compile.sparkle_dump!.to_smash
-          run_action 'Pre-processing template for graphing' do
+          run_action "Pre-processing template for graphing" do
             output_discovery(template_dump, @outputs, nil, nil)
-            ui.debug 'Output remapping results from pre-processing:'
+            ui.debug "Output remapping results from pre-processing:"
             @outputs.each_pair do |o_key, o_value|
               ui.debug "#{o_key} -> #{o_value}"
             end
             nil
           end
           graph = nil
-          run_action 'Generating resource graph' do
+          run_action "Generating resource graph" do
             graph = generate_graph(template_dump)
             nil
           end
-          run_action 'Writing graph result' do
+          run_action "Writing graph result" do
             FileUtils.mkdir_p(File.dirname(config[:output_file]))
-            if config[:output_type] == 'dot'
-              File.open("#{config[:output_file]}.dot", 'w') do |o_file|
+            if config[:output_type] == "dot"
+              File.open("#{config[:output_file]}.dot", "w") do |o_file|
                 o_file.puts graph.to_s
               end
             else
@@ -60,7 +60,7 @@ module Sfn
         else
           valid_providers = Provider.constants.sort.map { |provider|
             Bogo::Utility.snake(provider)
-          }.join('`, `')
+          }.join("`, `")
           ui.error "Graphing for provider `#{file.provider}` not currently supported."
           ui.error "Currently supported providers: `#{valid_providers}`."
         end
@@ -69,9 +69,9 @@ module Sfn
       def generate_graph(template, args = {})
         graph = ::Graph.new
         @root_graph = graph unless @root_graph
-        graph.graph_attribs << ::Graph::Attribute.new('overlap = false')
-        graph.graph_attribs << ::Graph::Attribute.new('splines = true')
-        graph.graph_attribs << ::Graph::Attribute.new('pack = true')
+        graph.graph_attribs << ::Graph::Attribute.new("overlap = false")
+        graph.graph_attribs << ::Graph::Attribute.new("splines = true")
+        graph.graph_attribs << ::Graph::Attribute.new("pack = true")
         graph.graph_attribs << ::Graph::Attribute.new('start = "random"')
         if args[:name]
           graph.name = "cluster_#{args[:name]}"
@@ -79,9 +79,9 @@ module Sfn
           graph.plaintext << graph.node(labelnode_key)
           graph.node(labelnode_key).label args[:name]
         else
-          graph.name = 'root'
+          graph.name = "root"
         end
-        edge_detection(template, graph, args[:name].to_s.sub('cluster_', ''), args.fetch(:resource_names, []))
+        edge_detection(template, graph, args[:name].to_s.sub("cluster_", ""), args.fetch(:resource_names, []))
         graph
       end
 
@@ -93,7 +93,7 @@ module Sfn
             memo + chr.ord
           end
         end
-        color = '#'
+        color = "#"
         3.times do |i|
           color << (255 ^ hash).to_s(16)
           new_val = hash + (hash * (1 / (i + 1.to_f))).to_i
@@ -108,12 +108,12 @@ module Sfn
 
       def validate_graph_style!
         if config[:luckymike]
-          ui.warn 'Detected luckymike power override. Forcing `dependency` style!'
-          config[:graph_style] = 'dependency'
+          ui.warn "Detected luckymike power override. Forcing `dependency` style!"
+          config[:graph_style] = "dependency"
         end
         config[:graph_style] = config[:graph_style].to_s
         unless GRAPH_STYLES.include?(config[:graph_style])
-          raise ArgumentError.new "Invalid graph style provided `#{config[:graph_style]}`. Valid: `#{GRAPH_STYLES.join('`, `')}`"
+          raise ArgumentError.new "Invalid graph style provided `#{config[:graph_style]}`. Valid: `#{GRAPH_STYLES.join("`, `")}`"
         end
       end
     end

data/lib/sfn/command/graph/aws.rb

@@ -1,4 +1,4 @@
-require 'sfn'
+require "sfn"
 
 module Sfn
   class Command
@@ -22,11 +22,11 @@ module Sfn
               k, v = hash.first
               if hash.size == 1
                 case k
-                when 'Ref'
+                when "Ref"
                   parameters.key?(v) ? parameters[v] : hash
-                when 'Fn::Join'
+                when "Fn::Join"
                   v.last
-                when 'Fn::Select'
+                when "Fn::Select"
                   v.last[v.first.to_i]
                 else
                   hash
@@ -37,25 +37,25 @@ module Sfn
             end
           end
 
-          def output_discovery(template, outputs, resource_name, parent_template, name = '')
-            if template['Resources']
-              template['Resources'].keys.each do |r_name|
-                r_info = template['Resources'][r_name]
-                if r_info['Type'] == 'AWS::CloudFormation::Stack'
-                  output_discovery(r_info['Properties']['Stack'], outputs, r_name, template, r_name)
+          def output_discovery(template, outputs, resource_name, parent_template, name = "")
+            if template["Resources"]
+              template["Resources"].keys.each do |r_name|
+                r_info = template["Resources"][r_name]
+                if r_info["Type"] == "AWS::CloudFormation::Stack"
+                  output_discovery(r_info["Properties"]["Stack"], outputs, r_name, template, r_name)
                 end
               end
             end
             if parent_template
               ui.debug "Pre-processing stack resource `#{resource_name}`"
               substack_parameters = Smash[
-                parent_template.fetch('Resources', resource_name, 'Properties', 'Parameters', {}).map do |key, value|
+                parent_template.fetch("Resources", resource_name, "Properties", "Parameters", {}).map do |key, value|
                   result = [key, value]
                   if value.is_a?(Hash)
                     v_key = value.keys.first
                     v_value = value.values.first
-                    if v_key == 'Fn::GetAtt' && parent_template.fetch('Resources', {}).keys.include?(v_value.first) && v_value.last.start_with?('Outputs.')
-                      output_key = v_value.first + '__' + v_value.last.split('.', 2).last
+                    if v_key == "Fn::GetAtt" && parent_template.fetch("Resources", {}).keys.include?(v_value.first) && v_value.last.start_with?("Outputs.")
+                      output_key = v_value.first + "__" + v_value.last.split(".", 2).last
                       ui.debug "Output key for check: #{output_key}"
                       if outputs.key?(output_key)
                         new_value = outputs[output_key]
@@ -72,33 +72,33 @@ module Sfn
 
               processor = AwsGraphProcessor.new({},
                                                 :parameters => substack_parameters)
-              template['Resources'] = processor.dereference_processor(
-                template['Resources'], ['Ref']
+              template["Resources"] = processor.dereference_processor(
+                template["Resources"], ["Ref"]
               )
-              template['Outputs'] = processor.dereference_processor(
-                template['Outputs'], ['Ref']
+              template["Outputs"] = processor.dereference_processor(
+                template["Outputs"], ["Ref"]
               )
               rename_processor = AwsGraphProcessor.new({},
                                                        :parameters => Smash[
-                                                         template.fetch('Resources', {}).keys.map do |r_key|
-                                                           [r_key, {'Ref' => [name, r_key].join}]
+                                                         template.fetch("Resources", {}).keys.map do |r_key|
+                                                           [r_key, {"Ref" => [name, r_key].join}]
                                                          end
                                                        ])
               derefed_outs = rename_processor.dereference_processor(
-                template.fetch('Outputs', {})
+                template.fetch("Outputs", {})
               ) || {}
 
               derefed_outs.each do |o_name, o_data|
-                o_key = [name, o_name].join('__')
-                outputs[o_key] = o_data['Value']
+                o_key = [name, o_name].join("__")
+                outputs[o_key] = o_data["Value"]
               end
             end
             outputs.dup.each do |key, value|
               if value.is_a?(Hash)
                 v_key = value.keys.first
                 v_value = value.values.first
-                if v_key == 'Fn::GetAtt' && v_value.last.start_with?('Outputs.')
-                  output_key = v_value.first << '__' << v_value.last.split('.', 2).last
+                if v_key == "Fn::GetAtt" && v_value.last.start_with?("Outputs.")
+                  output_key = v_value.first << "__" << v_value.last.split(".", 2).last
                   if outputs.key?(output_key)
                     outputs[key] = outputs[output_key]
                   end
@@ -107,16 +107,16 @@ module Sfn
             end
           end
 
-          def edge_detection(template, graph, name = '', resource_names = [])
-            resources = template.fetch('Resources', {})
+          def edge_detection(template, graph, name = "", resource_names = [])
+            resources = template.fetch("Resources", {})
             node_prefix = name
             resources.each do |resource_name, resource_data|
               node_name = [node_prefix, resource_name].join
-              if resource_data['Type'] == 'AWS::CloudFormation::Stack'
+              if resource_data["Type"] == "AWS::CloudFormation::Stack"
                 graph.subgraph << generate_graph(
-                  resource_data['Properties'].delete('Stack'),
+                  resource_data["Properties"].delete("Stack"),
                   :name => resource_name,
-                  :type => resource_data['Type'],
+                  :type => resource_data["Type"],
                   :resource_names => resource_names,
                 )
                 next
@@ -125,12 +125,12 @@ module Sfn
                 graph.box3d << graph.node(node_name)
               end
               graph.filled << graph.node(node_name)
-              graph.node(node_name).label "#{resource_name}\n<#{resource_data['Type']}>\n#{name}"
+              graph.node(node_name).label "#{resource_name}\n<#{resource_data["Type"]}>\n#{name}"
               resource_dependencies(resource_data, resource_names + resources.keys).each do |dep_name|
                 if resources.keys.include?(dep_name)
                   dep_name = [node_prefix, dep_name].join
                 end
-                if config[:graph_style] == 'creation'
+                if config[:graph_style] == "creation"
                   @root_graph.edge(dep_name, node_name)
                 else
                   @root_graph.edge(node_name, dep_name)
@@ -144,13 +144,13 @@ module Sfn
             case data
             when Hash
               data.map do |key, value|
-                if key == 'Ref' && names.include?(value)
+                if key == "Ref" && names.include?(value)
                   value
-                elsif key == 'DependsOn'
+                elsif key == "DependsOn"
                   [value].flatten.compact.find_all do |dependson_name|
                     names.include?(dependson_name)
                   end
-                elsif key == 'Fn::GetAtt' && names.include?(res = [value].flatten.compact.first)
+                elsif key == "Fn::GetAtt" && names.include?(res = [value].flatten.compact.first)
                   res
                 else
                   resource_dependencies(key, names) +

data/lib/sfn/command/graph/provider.rb

@@ -1,4 +1,4 @@
-require 'sfn'
+require "sfn"
 
 module Sfn
   class Command

data/lib/sfn/command/graph/terraform.rb

@@ -1,4 +1,4 @@
-require 'sfn'
+require "sfn"
 
 module Sfn
   class Command
@@ -36,18 +36,18 @@ module Sfn
 
             def parameters
               Hash[
-                @original.fetch('parameters', {}).map do |k, v|
-                  [k, v.fetch('default', '')]
+                @original.fetch("parameters", {}).map do |k, v|
+                  [k, v.fetch("default", "")]
                 end
               ].merge(@parameters)
             end
 
             def resources
-              @original.fetch('resources', {})
+              @original.fetch("resources", {})
             end
 
             def outputs
-              @original.fetch('outputs', {})
+              @original.fetch("outputs", {})
             end
 
             def apply_function(string, funcs = [])
@@ -61,22 +61,22 @@ module Sfn
             end
           end
 
-          def output_discovery(template, outputs, resource_name, parent_template, name = '')
-            if template['resources']
-              template['resources'].keys.each do |r_name|
-                r_info = template['resources'][r_name]
-                if r_info['type'] == 'module'
-                  output_discovery(r_info['properties']['stack'], outputs, r_name, template, r_name)
+          def output_discovery(template, outputs, resource_name, parent_template, name = "")
+            if template["resources"]
+              template["resources"].keys.each do |r_name|
+                r_info = template["resources"][r_name]
+                if r_info["type"] == "module"
+                  output_discovery(r_info["properties"]["stack"], outputs, r_name, template, r_name)
                 end
               end
             end
             if parent_template
               ui.debug "Pre-processing stack resource `#{resource_name}`"
               substack_parameters = Smash[
-                parent_template.fetch('resources', resource_name, 'properties', 'parameters', {}).map do |key, value|
+                parent_template.fetch("resources", resource_name, "properties", "parameters", {}).map do |key, value|
                   result = [key, value]
-                  if value.to_s.start_with?('${module.')
-                    output_key = value.sub('${module.', '').sub('}', '').sub('.', '__')
+                  if value.to_s.start_with?("${module.")
+                    output_key = value.sub("${module.", "").sub("}", "").sub(".", "__")
                     ui.debug "Output key for check: #{output_key}"
                     if outputs.key?(output_key)
                       new_value = outputs[output_key]
@@ -92,27 +92,27 @@ module Sfn
 
               processor = TerraformGraphProcessor.new({},
                                                       :parameters => substack_parameters)
-              template['resources'] = processor.dereference_processor(
-                template['resources'], []
+              template["resources"] = processor.dereference_processor(
+                template["resources"], []
               )
-              template['outputs'] = processor.dereference_processor(
-                template['outputs'], []
+              template["outputs"] = processor.dereference_processor(
+                template["outputs"], []
               )
-              derefed_outs = template['outputs'] || {}
+              derefed_outs = template["outputs"] || {}
               derefed_outs.each do |o_name, o_data|
-                o_key = [name, o_name].join('__')
-                val = o_data['value']
-                if val.start_with?('${') && val.scan('.').count == 2
-                  val = val.split('.')
+                o_key = [name, o_name].join("__")
+                val = o_data["value"]
+                if val.start_with?("${") && val.scan(".").count == 2
+                  val = val.split(".")
                   val[1] = "#{name}__#{val[1]}"
-                  val = val.join('.')
+                  val = val.join(".")
                 end
                 outputs[o_key] = val
               end
             end
             outputs.dup.each do |key, value|
-              if value.to_s.start_with?('${module.')
-                output_key = value.to_s.sub('${module.', '').sub('}', '').sub('.', '__')
+              if value.to_s.start_with?("${module.")
+                output_key = value.to_s.sub("${module.", "").sub("}", "").sub(".", "__")
                 if outputs.key?(output_key)
                   outputs[key] = outputs[output_key]
                 end
@@ -120,16 +120,16 @@ module Sfn
             end
           end
 
-          def edge_detection(template, graph, name = '', resource_names = [])
-            resources = (template.fetch('resources', {}) || {})
+          def edge_detection(template, graph, name = "", resource_names = [])
+            resources = (template.fetch("resources", {}) || {})
             node_prefix = name
             resources.each do |resource_name, resource_data|
-              node_name = [node_prefix, resource_name].join('__')
-              if resource_data['type'] == 'module'
+              node_name = [node_prefix, resource_name].join("__")
+              if resource_data["type"] == "module"
                 graph.subgraph << generate_graph(
-                  resource_data['properties'].delete('stack'),
+                  resource_data["properties"].delete("stack"),
                   :name => resource_name,
-                  :type => resource_data['type'],
+                  :type => resource_data["type"],
                   :resource_names => resource_names,
                 )
                 next
@@ -138,28 +138,28 @@ module Sfn
                 graph.box3d << graph.node(node_name)
               end
               graph.filled << graph.node(node_name)
-              graph.node(node_name).label "#{resource_name}\n<#{resource_data['type']}>\n#{name}"
+              graph.node(node_name).label "#{resource_name}\n<#{resource_data["type"]}>\n#{name}"
               resource_dependencies(resource_data, resource_names + resources.keys).each do |dep_name|
                 if resources.keys.include?(dep_name)
-                  dep_name = [node_prefix, dep_name].join('__')
+                  dep_name = [node_prefix, dep_name].join("__")
                 end
-                if config[:graph_style] == 'creation'
+                if config[:graph_style] == "creation"
                   @root_graph.edge(dep_name, node_name)
                 else
                   @root_graph.edge(node_name, dep_name)
                 end
               end
             end
-            resource_names.concat resources.keys.map { |r_name| [node_prefix, r_name].join('__') }
+            resource_names.concat resources.keys.map { |r_name| [node_prefix, r_name].join("__") }
           end
 
           def resource_dependencies(data, names)
             case data
             when String
               result = []
-              if data.start_with?('${') && data.scan('.').count >= 1
-                data = data.tr('${}', '')
-                check_name = data.split('.')[1]
+              if data.start_with?("${") && data.scan(".").count >= 1
+                data = data.tr("${}", "")
+                check_name = data.split(".")[1]
                 if names.include?(check_name)
                   result.push(check_name)
                 end
@@ -167,9 +167,9 @@ module Sfn
               result
             when Hash
               data.map do |key, value|
-                if key == 'depends_on'
+                if key == "depends_on"
                   [value].flatten.compact.map do |dependson_name|
-                    dep_name = dependson_name.split('.').last
+                    dep_name = dependson_name.split(".").last
                     dep_name if names.include?(dep_name)
                   end
                 else