chef 14.6.47-universal-mingw32 → 14.7.17-universal-mingw32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 590192dd2bab015780a7b514985983f269e184026100a6a87d147980d7f95708
-  data.tar.gz: 07c83917cd5c5ed3f53c76c43b5eca7a904d6c65c3173f5af4267a4d0a96c531
+  metadata.gz: 9be735cf1e06242e8769af292fd4fdf773e45d9a63c3590474b5402b592cd5a2
+  data.tar.gz: 409f58ef97a76f840f9b062bbe6a695c2e483fe3f485571d4a9f390b61c2c9c7
 SHA512:
-  metadata.gz: 1c44fee67b35e960b6897ee3b540e341f7231af12b44df1fcb95dd270c8bfe5a56bf7784b627a6516439c3e115e1c7bc31550c0e5c23e83f2ab8e81ea1c7b789
-  data.tar.gz: fc1ccdbec5a77be37902a7d1c4c9bb69545e11416f30379351ea064f1e95aae885a7f13d256ccf05770b3f32a9d441c0130d9e1d48430abdcceff50a82fcc426
+  metadata.gz: 03f823b4b540570f1f0f6814952f2fff5d2c3f3e2e3d4366eecc0d0a5569e9ee0c436bbff27b34f865cdd6435b59591a1a87ad91eb925c820b8bbcab01f9d55f
+  data.tar.gz: '09b67f8b5ffe3caff4c45ac9d388bac6fbb26ce50e4a9abf32961a84e331caaa2f92f8d1b5ff979dbdcac741b605b0f4a5b5825b1c4d53548eb1c88b1d3eb1b4'

data/chef-universal-mingw32.gemspec

@@ -16,6 +16,7 @@ gemspec.add_dependency "windows-api", "~> 0.4.4"
 gemspec.add_dependency "wmi-lite", "~> 1.0"
 gemspec.add_dependency "win32-taskscheduler", "~> 2.0"
 gemspec.add_dependency "iso8601", "~> 0.12.1"
+gemspec.add_dependency "win32-certstore", ">= 0.1.8"
 gemspec.extensions << "ext/win32-eventlog/Rakefile"
 gemspec.files += Dir.glob("{distro,ext}/**/*")
 

data/lib/chef/application/apply.rb

@@ -191,10 +191,12 @@ class Chef::Application::Apply < Chef::Application
     recipe, run_context = get_recipe_and_run_context
     recipe.instance_eval(@recipe_text, @recipe_filename, 1)
     runner = Chef::Runner.new(run_context)
-    begin
-      runner.converge
-    ensure
-      @recipe_fh.close
+    catch(:end_client_run_early) do
+      begin
+        runner.converge
+      ensure
+        @recipe_fh.close
+      end
     end
     Chef::Platform::Rebooter.reboot_if_needed!(runner)
   end

data/lib/chef/knife/config_list_profiles.rb

@@ -21,7 +21,7 @@ require "chef/workstation_config_loader"
 class Chef
   class Knife
     class ConfigListProfiles < Knife
-      banner "knife config list-profiles"
+      banner "knife config list-profiles (options)"
 
       option :ignore_knife_rb,
              short: "-i",

data/lib/chef/knife/list.rb

@@ -19,7 +19,7 @@ require "chef/chef_fs/knife"
 class Chef
   class Knife
     class List < Chef::ChefFS::Knife
-      banner "knife list [-dfR1p] [PATTERN1 ... PATTERNn]"
+      banner "knife list [-dfR1p] [PATTERN1 ... PATTERNn] (options)"
 
       category "path-based"
 

data/lib/chef/knife/raw.rb

@@ -20,7 +20,7 @@ require "chef/http"
 class Chef
   class Knife
     class Raw < Chef::Knife
-      banner "knife raw REQUEST_PATH"
+      banner "knife raw REQUEST_PATH (options)"
 
       deps do
         require "chef/json_compat"

data/lib/chef/knife/role_env_run_list_clear.rb

@@ -28,7 +28,7 @@ class Chef
         require "chef/json_compat"
       end
 
-      banner "knife role env_run_list clear [ROLE] [ENVIRONMENT]"
+      banner "knife role env_run_list clear [ROLE] [ENVIRONMENT] (options)"
       def clear_env_run_list(role, environment)
         nlist = []
         role.env_run_lists_add(environment => nlist)

data/lib/chef/knife/role_env_run_list_remove.rb

@@ -27,7 +27,7 @@ class Chef
         require "chef/json_compat"
       end
 
-      banner "knife role env_run_list remove [ROLE] [ENVIRONMENT] [ENTRIES]"
+      banner "knife role env_run_list remove [ROLE] [ENVIRONMENT] [ENTRIES] (options)"
 
       def remove_from_env_run_list(role, environment, item_to_remove)
         nlist = []

data/lib/chef/knife/role_env_run_list_replace.rb

@@ -28,7 +28,7 @@ class Chef
         require "chef/json_compat"
       end
 
-      banner "knife role env_run_list replace [ROLE] [ENVIRONMENT] [OLD_ENTRY] [NEW_ENTRY] "
+      banner "knife role env_run_list replace [ROLE] [ENVIRONMENT] [OLD_ENTRY] [NEW_ENTRY] (options)"
 
       def replace_in_env_run_list(role, environment, old_entry, new_entry)
         nlist = []

data/lib/chef/knife/role_env_run_list_set.rb

@@ -28,7 +28,7 @@ class Chef
         require "chef/json_compat"
       end
 
-      banner "knife role env_run_list set [ROLE] [ENVIRONMENT] [ENTRIES]"
+      banner "knife role env_run_list set [ROLE] [ENVIRONMENT] [ENTRIES] (optionss)"
 
       # Clears out any existing env_run_list_items and sets them to the
       # specified entries

data/lib/chef/knife/role_run_list_clear.rb

@@ -28,7 +28,7 @@ class Chef
         require "chef/json_compat"
       end
 
-      banner "knife role run_list clear [ROLE]"
+      banner "knife role run_list clear [ROLE] (options)"
       def clear_env_run_list(role, environment)
         nlist = []
         role.env_run_lists_add(environment => nlist)

data/lib/chef/knife/role_run_list_remove.rb

@@ -26,7 +26,7 @@ class Chef
         require "chef/role"
       end
 
-      banner "knife role run_list remove [ROLE] [ENTRY]"
+      banner "knife role run_list remove [ROLE] [ENTRY] (options)"
 
       def remove_from_env_run_list(role, environment, item_to_remove)
         nlist = []

data/lib/chef/knife/role_run_list_replace.rb

@@ -28,7 +28,7 @@ class Chef
         require "chef/json_compat"
       end
 
-      banner "knife role run_list replace [ROLE] [OLD_ENTRY] [NEW_ENTRY] "
+      banner "knife role run_list replace [ROLE] [OLD_ENTRY] [NEW_ENTRY] (options)"
 
       def replace_in_env_run_list(role, environment, old_entry, new_entry)
         nlist = []

data/lib/chef/knife/role_run_list_set.rb

@@ -27,7 +27,7 @@ class Chef
         require "chef/role"
       end
 
-      banner "knife role run_list set [ROLE] [ENTRIES]"
+      banner "knife role run_list set [ROLE] [ENTRIES] (options)"
 
       # Clears out any existing env_run_list_items and sets them to the
       # specified entries

data/lib/chef/knife/show.rb

@@ -19,7 +19,7 @@ require "chef/chef_fs/knife"
 class Chef
   class Knife
     class Show < Chef::ChefFS::Knife
-      banner "knife show [PATTERN1 ... PATTERNn]"
+      banner "knife show [PATTERN1 ... PATTERNn] (options)"
 
       category "path-based"
 

data/lib/chef/knife/upload.rb

@@ -19,7 +19,7 @@ require "chef/chef_fs/knife"
 class Chef
   class Knife
     class Upload < Chef::ChefFS::Knife
-      banner "knife upload PATTERNS"
+      banner "knife upload PATTERNS (options)"
 
       category "path-based"
 

data/lib/chef/knife/xargs.rb

@@ -19,7 +19,7 @@ require "chef/chef_fs/knife"
 class Chef
   class Knife
     class Xargs < Chef::ChefFS::Knife
-      banner "knife xargs [COMMAND]"
+      banner "knife xargs [COMMAND] (options)"
 
       category "path-based"
 

data/lib/chef/node/attribute.rb

@@ -78,113 +78,117 @@ class Chef
         :@force_override,
       ].freeze
 
-      [:all?,
-       :any?,
-       :assoc,
-       :chunk,
-       :collect,
-       :collect_concat,
-       :compare_by_identity,
-       :compare_by_identity?,
-       :count,
-       :cycle,
-       :detect,
-       :drop,
-       :drop_while,
-       :each,
-       :each_cons,
-       :each_entry,
-       :each_key,
-       :each_pair,
-       :each_slice,
-       :each_value,
-       :each_with_index,
-       :each_with_object,
-       :empty?,
-       :entries,
-       :except,
-       :fetch,
-       :find,
-       :find_all,
-       :find_index,
-       :first,
-       :flat_map,
-       :flatten,
-       :grep,
-       :group_by,
-       :has_value?,
-       :include?,
-       :index,
-       :inject,
-       :invert,
-       :key,
-       :keys,
-       :length,
-       :map,
-       :max,
-       :max_by,
-       :merge,
-       :min,
-       :min_by,
-       :minmax,
-       :minmax_by,
-       :none?,
-       :one?,
-       :partition,
-       :rassoc,
-       :reduce,
-       :reject,
-       :reverse_each,
-       :select,
-       :size,
-       :slice_before,
-       :sort,
-       :sort_by,
-       :store,
-       :symbolize_keys,
-       :take,
-       :take_while,
-       :to_a,
-       :to_h,
-       :to_hash,
-       :to_set,
-       :value?,
-       :values,
-       :values_at,
-       :zip].each do |delegated_method|
-         define_method(delegated_method) do |*args, &block|
-           merged_attributes.send(delegated_method, *args, &block)
-         end
-       end
-
-       # return the cookbook level default attribute component
+      ENUM_METHODS = [
+        :all?,
+        :any?,
+        :assoc,
+        :chunk,
+        :collect,
+        :collect_concat,
+        :compare_by_identity,
+        :compare_by_identity?,
+        :count,
+        :cycle,
+        :detect,
+        :drop,
+        :drop_while,
+        :each,
+        :each_cons,
+        :each_entry,
+        :each_key,
+        :each_pair,
+        :each_slice,
+        :each_value,
+        :each_with_index,
+        :each_with_object,
+        :empty?,
+        :entries,
+        :except,
+        :fetch,
+        :find,
+        :find_all,
+        :find_index,
+        :first,
+        :flat_map,
+        :flatten,
+        :grep,
+        :group_by,
+        :has_value?,
+        :include?,
+        :index,
+        :inject,
+        :invert,
+        :key,
+        :keys,
+        :length,
+        :map,
+        :max,
+        :max_by,
+        :merge,
+        :min,
+        :min_by,
+        :minmax,
+        :minmax_by,
+        :none?,
+        :one?,
+        :partition,
+        :rassoc,
+        :reduce,
+        :reject,
+        :reverse_each,
+        :select,
+        :size,
+        :slice_before,
+        :sort,
+        :sort_by,
+        :store,
+        :symbolize_keys,
+        :take,
+        :take_while,
+        :to_a,
+        :to_h,
+        :to_hash,
+        :to_set,
+        :value?,
+        :values,
+        :values_at,
+        :zip,
+      ].freeze
+
+      ENUM_METHODS.each do |delegated_method|
+        define_method(delegated_method) do |*args, &block|
+          merged_attributes.send(delegated_method, *args, &block)
+        end
+      end
+
+      # return the cookbook level default attribute component
       attr_reader :default
 
-       # return the role level default attribute component
+      # return the role level default attribute component
       attr_reader :role_default
 
-       # return the environment level default attribute component
+      # return the environment level default attribute component
       attr_reader :env_default
 
-       # return the force_default level attribute component
+      # return the force_default level attribute component
       attr_reader :force_default
 
-       # return the "normal" level attribute component
+      # return the "normal" level attribute component
       attr_reader :normal
 
-       # return the cookbook level override attribute component
+      # return the cookbook level override attribute component
       attr_reader :override
 
-       # return the role level override attribute component
+      # return the role level override attribute component
       attr_reader :role_override
 
-       # return the enviroment level override attribute component
+      # return the enviroment level override attribute component
       attr_reader :env_override
 
-       # return the force override level attribute component
+      # return the force override level attribute component
       attr_reader :force_override
 
-       # return the automatic level attribute component
+      # return the automatic level attribute component
       attr_reader :automatic
 
       def initialize(normal, default, override, automatic, node = nil)
@@ -205,15 +209,15 @@ class Chef
         super(nil, self, node, :merged)
       end
 
-       # Debug what's going on with an attribute. +args+ is a path spec to the
-       # attribute you're interested in. For example, to debug where the value
-       # of `node[:network][:default_interface]` is coming from, use:
-       #   debug_value(:network, :default_interface).
-       # The return value is an Array of Arrays.  The Arrays
-       # are pairs of `["precedence_level", value]`, where precedence level is
-       # the component, such as role default, normal, etc. and value is the
-       # attribute value set at that precedence level. If there is no value at
-       # that precedence level, +value+ will be the symbol +:not_present+.
+      # Debug what's going on with an attribute. +args+ is a path spec to the
+      # attribute you're interested in. For example, to debug where the value
+      # of `node[:network][:default_interface]` is coming from, use:
+      #   debug_value(:network, :default_interface).
+      # The return value is an Array of Arrays.  The Arrays
+      # are pairs of `["precedence_level", value]`, where precedence level is
+      # the component, such as role default, normal, etc. and value is the
+      # attribute value set at that precedence level. If there is no value at
+      # that precedence level, +value+ will be the symbol +:not_present+.
       def debug_value(*args)
         COMPONENTS.map do |component|
           value =
@@ -226,49 +230,49 @@ class Chef
         end
       end
 
-       # Set the cookbook level default attribute component to +new_data+.
+      # Set the cookbook level default attribute component to +new_data+.
       def default=(new_data)
         reset
         @default = VividMash.new(new_data, self, __node__, :default)
       end
 
-       # Set the role level default attribute component to +new_data+
+      # Set the role level default attribute component to +new_data+
       def role_default=(new_data)
         reset
         @role_default = VividMash.new(new_data, self, __node__, :role_default)
       end
 
-       # Set the environment level default attribute component to +new_data+
+      # Set the environment level default attribute component to +new_data+
       def env_default=(new_data)
         reset
         @env_default = VividMash.new(new_data, self, __node__, :env_default)
       end
 
-       # Set the force_default (+default!+) level attributes to +new_data+
+      # Set the force_default (+default!+) level attributes to +new_data+
       def force_default=(new_data)
         reset
         @force_default = VividMash.new(new_data, self, __node__, :force_default)
       end
 
-       # Set the normal level attribute component to +new_data+
+      # Set the normal level attribute component to +new_data+
       def normal=(new_data)
         reset
         @normal = VividMash.new(new_data, self, __node__, :normal)
       end
 
-       # Set the cookbook level override attribute component to +new_data+
+      # Set the cookbook level override attribute component to +new_data+
       def override=(new_data)
         reset
         @override = VividMash.new(new_data, self, __node__, :override)
       end
 
-       # Set the role level override attribute component to +new_data+
+      # Set the role level override attribute component to +new_data+
       def role_override=(new_data)
         reset
         @role_override = VividMash.new(new_data, self, __node__, :role_override)
       end
 
-       # Set the environment level override attribute component to +new_data+
+      # Set the environment level override attribute component to +new_data+
       def env_override=(new_data)
         reset
         @env_override = VividMash.new(new_data, self, __node__, :env_override)
@@ -284,11 +288,11 @@ class Chef
         @automatic = VividMash.new(new_data, self, __node__, :automatic)
       end
 
-       #
-       # Deleting attributes
-       #
+      #
+      # Deleting attributes
+      #
 
-       # clears attributes from all precedence levels
+      # clears attributes from all precedence levels
       def rm(*args)
         with_deep_merged_return_value(self, *args) do
           rm_default(*args)
@@ -297,11 +301,11 @@ class Chef
         end
       end
 
-       # clears attributes from all default precedence levels
-       #
-       # similar to: force_default!['foo']['bar'].delete('baz')
-       # - does not autovivify
-       # - does not trainwreck if interior keys do not exist
+      # clears attributes from all default precedence levels
+      #
+      # similar to: force_default!['foo']['bar'].delete('baz')
+      # - does not autovivify
+      # - does not trainwreck if interior keys do not exist
       def rm_default(*args)
         with_deep_merged_return_value(combined_default, *args) do
           default.unlink(*args)
@@ -311,20 +315,20 @@ class Chef
         end
       end
 
-       # clears attributes from normal precedence
-       #
-       # equivalent to: normal!['foo']['bar'].delete('baz')
-       # - does not autovivify
-       # - does not trainwreck if interior keys do not exist
+      # clears attributes from normal precedence
+      #
+      # equivalent to: normal!['foo']['bar'].delete('baz')
+      # - does not autovivify
+      # - does not trainwreck if interior keys do not exist
       def rm_normal(*args)
         normal.unlink(*args)
       end
 
-       # clears attributes from all override precedence levels
-       #
-       # equivalent to: force_override!['foo']['bar'].delete('baz')
-       # - does not autovivify
-       # - does not trainwreck if interior keys do not exist
+      # clears attributes from all override precedence levels
+      #
+      # equivalent to: force_override!['foo']['bar'].delete('baz')
+      # - does not autovivify
+      # - does not trainwreck if interior keys do not exist
       def rm_override(*args)
         with_deep_merged_return_value(combined_override, *args) do
           override.unlink(*args)
@@ -344,37 +348,37 @@ class Chef
 
       private :with_deep_merged_return_value
 
-       #
-       # Replacing attributes without merging
-       #
+      #
+      # Replacing attributes without merging
+      #
 
-       # sets default attributes without merging
-       #
-       # - this API autovivifies (and cannot trainwreck)
+      # sets default attributes without merging
+      #
+      # - this API autovivifies (and cannot trainwreck)
       def default!(*args)
         return Decorator::Unchain.new(self, :default!) unless args.length > 0
         write(:default, *args)
       end
 
-       # sets normal attributes without merging
-       #
-       # - this API autovivifies (and cannot trainwreck)
+      # sets normal attributes without merging
+      #
+      # - this API autovivifies (and cannot trainwreck)
       def normal!(*args)
         return Decorator::Unchain.new(self, :normal!) unless args.length > 0
         write(:normal, *args)
       end
 
-       # sets override attributes without merging
-       #
-       # - this API autovivifies (and cannot trainwreck)
+      # sets override attributes without merging
+      #
+      # - this API autovivifies (and cannot trainwreck)
       def override!(*args)
         return Decorator::Unchain.new(self, :override!) unless args.length > 0
         write(:override, *args)
       end
 
-       # clears from all default precedence levels and then sets force_default
-       #
-       # - this API autovivifies (and cannot trainwreck)
+      # clears from all default precedence levels and then sets force_default
+      #
+      # - this API autovivifies (and cannot trainwreck)
       def force_default!(*args)
         return Decorator::Unchain.new(self, :force_default!) unless args.length > 0
         value = args.pop
@@ -382,7 +386,7 @@ class Chef
         write(:force_default, *args, value)
       end
 
-       # clears from all override precedence levels and then sets force_override
+      # clears from all override precedence levels and then sets force_override
       def force_override!(*args)
         return Decorator::Unchain.new(self, :force_override!) unless args.length > 0
         value = args.pop
@@ -390,14 +394,13 @@ class Chef
         write(:force_override, *args, value)
       end
 
-       #
-       # Accessing merged attributes.
-       #
-       # Note that merged_attributes('foo', 'bar', 'baz') can be called to compute only the
-       # deep merge of node['foo']['bar']['baz'], but in practice we currently always compute
-       # all of node['foo'] even if the user only requires node['foo']['bar']['baz'].
-       #
-
+      #
+      # Accessing merged attributes.
+      #
+      # Note that merged_attributes('foo', 'bar', 'baz') can be called to compute only the
+      # deep merge of node['foo']['bar']['baz'], but in practice we currently always compute
+      # all of node['foo'] even if the user only requires node['foo']['bar']['baz'].
+      #
       def merged_attributes(*path)
         merge_all(path)
       end
@@ -430,7 +433,8 @@ class Chef
           instance_variable_get(component_ivar).key?(key)
         end
       end
-       # method-style access to attributes (has to come after the prepended ImmutablizeHash)
+
+      # method-style access to attributes (has to come after the prepended ImmutablizeHash)
 
       def read(*path)
         merged_attributes.read(*path)
@@ -479,16 +483,16 @@ class Chef
 
       private
 
-       # Helper method for merge_all/merge_defaults/merge_overrides.
-       #
-       # apply_path(thing, [ "foo", "bar", "baz" ]) = thing["foo"]["bar"]["baz"]
-       #
-       # The path value can be nil in which case the whole component is returned.
-       #
-       # It returns nil (does not raise an exception) if it walks off the end of an Mash/Hash/Array, it does not
-       # raise any TypeError if it attempts to apply a hash key to an Integer/String/TrueClass, and just returns
-       # nil in that case.
-       #
+      # Helper method for merge_all/merge_defaults/merge_overrides.
+      #
+      # apply_path(thing, [ "foo", "bar", "baz" ]) = thing["foo"]["bar"]["baz"]
+      #
+      # The path value can be nil in which case the whole component is returned.
+      #
+      # It returns nil (does not raise an exception) if it walks off the end of an Mash/Hash/Array, it does not
+      # raise any TypeError if it attempts to apply a hash key to an Integer/String/TrueClass, and just returns
+      # nil in that case.
+      #
       def apply_path(component, path)
         path ||= []
         path.inject(component) do |val, path_arg|
@@ -509,20 +513,20 @@ class Chef
         end
       end
 
-       # For elements like Fixnums, true, nil...
+      # For elements like Fixnums, true, nil...
       def safe_dup(e)
         e.dup
       rescue TypeError
         e
       end
 
-       # Deep merge all attribute levels using hash-only merging between different precidence
-       # levels (so override arrays completely replace arrays set at any default level).
-       #
-       # The path allows for selectively deep-merging a subtree of the node object.
-       #
-       # @param path [Array] Array of args to method chain to descend into the node object
-       # @return [attr] Deep Merged values (may be VividMash, Hash, Array, etc) from the node object
+      # Deep merge all attribute levels using hash-only merging between different precidence
+      # levels (so override arrays completely replace arrays set at any default level).
+      #
+      # The path allows for selectively deep-merging a subtree of the node object.
+      #
+      # @param path [Array] Array of args to method chain to descend into the node object
+      # @return [attr] Deep Merged values (may be VividMash, Hash, Array, etc) from the node object
       def merge_all(path)
         components = [
           merge_defaults(path),
@@ -537,12 +541,12 @@ class Chef
         ret == NIL ? nil : ret
       end
 
-       # Deep merge the default attribute levels with array merging.
-       #
-       # The path allows for selectively deep-merging a subtree of the node object.
-       #
-       # @param path [Array] Array of args to method chain to descend into the node object
-       # @return [attr] Deep Merged values (may be VividMash, Hash, Array, etc) from the node object
+      # Deep merge the default attribute levels with array merging.
+      #
+      # The path allows for selectively deep-merging a subtree of the node object.
+      #
+      # @param path [Array] Array of args to method chain to descend into the node object
+      # @return [attr] Deep Merged values (may be VividMash, Hash, Array, etc) from the node object
       def merge_defaults(path)
         ret = DEFAULT_COMPONENTS.inject(NIL) do |merged, component_ivar|
           component_value = apply_path(instance_variable_get(component_ivar), path)
@@ -551,12 +555,12 @@ class Chef
         ret == NIL ? nil : ret
       end
 
-       # Deep merge the override attribute levels with array merging.
-       #
-       # The path allows for selectively deep-merging a subtree of the node object.
-       #
-       # @param path [Array] Array of args to method chain to descend into the node object
-       # @return [attr] Deep Merged values (may be VividMash, Hash, Array, etc) from the node object
+      # Deep merge the override attribute levels with array merging.
+      #
+      # The path allows for selectively deep-merging a subtree of the node object.
+      #
+      # @param path [Array] Array of args to method chain to descend into the node object
+      # @return [attr] Deep Merged values (may be VividMash, Hash, Array, etc) from the node object
       def merge_overrides(path)
         ret = OVERRIDE_COMPONENTS.inject(NIL) do |merged, component_ivar|
           component_value = apply_path(instance_variable_get(component_ivar), path)
@@ -565,7 +569,7 @@ class Chef
         ret == NIL ? nil : ret
       end
 
-       # needed for __path__
+      # needed for __path__
       def convert_key(key)
         key.kind_of?(Symbol) ? key.to_s : key
       end
@@ -592,11 +596,11 @@ class Chef
         elsif merge_onto.kind_of?(Array) && merge_with.kind_of?(Array)
           merge_onto |= merge_with
 
-        # If merge_with is nil, don't replace merge_onto
+          # If merge_with is nil, don't replace merge_onto
         elsif merge_with.nil?
           merge_onto
 
-        # In all other cases, replace merge_onto with merge_with
+          # In all other cases, replace merge_onto with merge_with
         else
           if merge_with.kind_of?(Hash)
             Chef::Node::ImmutableMash.new(merge_with)
@@ -625,11 +629,11 @@ class Chef
           end
           merge_onto
 
-        # If merge_with is nil, don't replace merge_onto
+          # If merge_with is nil, don't replace merge_onto
         elsif merge_with.nil?
           merge_onto
 
-        # In all other cases, replace merge_onto with merge_with
+          # In all other cases, replace merge_onto with merge_with
         else
           if merge_with.kind_of?(Hash)
             Chef::Node::ImmutableMash.new(merge_with)