Fix for #4167 -- overriding file permissions in conf file

The logic which iterates over the searchpath in reverse does not translate the name.
Therefore file overrides in :master or :agent are not picked up.

2 files changed, 2 insertions, 1 deletions

diff --git a/lib/puppet/util/settings.rb b/lib/puppet/util/settings.rb
index e1f7c3578..992371367 100644
--- a/lib/puppet/util/settings.rb
+++ b/lib/puppet/util/settings.rb
@@ -364,6 +364,7 @@ class Puppet::Util::Settings
         # because multiple sections could set the same value
         # and I'm too lazy to only set the metadata once.
         searchpath.reverse.each do |source|
+            source = run_mode if source == :run_mode
             if meta = metas[source]
                 set_metadata(meta)
             end
diff --git a/lib/puppet/util/settings/file_setting.rb b/lib/puppet/util/settings/file_setting.rb
index 732af0b09..6f0f315eb 100644
--- a/lib/puppet/util/settings/file_setting.rb
+++ b/lib/puppet/util/settings/file_setting.rb
@@ -3,7 +3,7 @@ require 'puppet/util/settings/setting'
 # A file.
 class Puppet::Util::Settings::FileSetting < Puppet::Util::Settings::Setting
     AllowedOwners = %w{root service}
-    AllowedGroups = %w{service}
+    AllowedGroups = %w{root service}
 
     class SettingError < StandardError; end