fix sections settings for various configuration params

Author: Benjamin Krein

manifests/master.pp

@@ -217,6 +217,7 @@
     ensure  => $setting_config,
     setting => 'modulepath',
     value   => $modulepath,
+    section => 'main',
   }
   ini_setting {'puppetmastermanifest':
     ensure  => $setting_config,
@@ -227,6 +228,7 @@
     ensure  => $setting_directory,
     setting => 'environmentpath',
     value   => $environmentpath,
+    section => 'main',
   }
 
   if $external_nodes != undef {
@@ -278,6 +280,7 @@
     ensure  => present,
     setting => 'pluginsync',
     value   => $pluginsync,
+    section => 'agent',
   }
 
   ini_setting {'puppetmasterparser':
@@ -304,6 +307,7 @@
       ensure  => present,
       setting => 'digest_algorithm',
       value   => $digest_algorithm,
+      section => 'main',
   }
 
   anchor { 'puppet::master::end': }

spec/classes/puppet_master_spec.rb

@@ -82,7 +82,7 @@
             )
             should contain_ini_setting('puppetmastermodulepath').with(
                 :ensure  => 'present',
-                :section => 'master',
+                :section => 'main',
                 :setting => 'modulepath',
                 :path    => '/etc/puppet/puppet.conf',
                 :value   => params[:modulepath],
@@ -146,7 +146,7 @@
             )
             should contain_ini_setting('puppetmasterpluginsync').with(
                 :ensure  => 'present',
-                :section => 'master',
+                :section => 'agent',
                 :setting => 'pluginsync',
                 :path    => '/etc/puppet/puppet.conf',
                 :value   => 'true'
@@ -243,7 +243,7 @@
             )
             should contain_ini_setting('puppetmastermodulepath').with(
                 :ensure  => 'present',
-                :section => 'master',
+                :section => 'main',
                 :setting => 'modulepath',
                 :path    => '/etc/puppet/puppet.conf',
                 :value   => params[:modulepath],
@@ -307,7 +307,7 @@
             )
             should contain_ini_setting('puppetmasterpluginsync').with(
                 :ensure  => 'present',
-                :section => 'master',
+                :section => 'agent',
                 :setting => 'pluginsync',
                 :path    => '/etc/puppet/puppet.conf',
                 :value   => 'true'
@@ -342,7 +342,7 @@
         it {
             should contain_ini_setting('puppetmasterenvironmentpath').with(
                 :ensure  => 'present',
-                :section => 'master',
+                :section => 'main',
                 :setting => 'environmentpath',
                 :path    => '/etc/puppet/puppet.conf',
                 :value   => '$confdir/environments'
@@ -375,7 +375,7 @@
         it {
             should contain_ini_setting('puppetmasterenvironmentpath').with(
                 :ensure  => 'present',
-                :section => 'master',
+                :section => 'main',
                 :setting => 'environmentpath',
                 :path    => '/etc/puppet/puppet.conf',
                 :value   => '/etc/puppetlabs/puppet/environments'