Allow reports to be disabled

manifests/agent/config.pp

@@ -4,7 +4,7 @@
     'classfile':         value => $::puppet::classfile;
     'localconfig':       value => '$vardir/localconfig';
     'default_schedules': value => false;
-    'report':            value => true;
+    'report':            value => $::puppet::report;
     'pluginsync':        value => $::puppet::pluginsync;
     'masterport':        value => $::puppet::port;
     'environment':       value => $::puppet::environment;

manifests/init.pp

@@ -199,6 +199,8 @@
 # $dir_group::                              Group of the base puppet directory, used when
 #                                           puppet::server is false.
 #
+# $report::                                 Send reports to the Puppet Master
+#
 # == puppet::server parameters
 #
 # $server::                                 Should a puppet master be installed as well as the client
@@ -608,6 +610,7 @@
   Variant[String, Array[String]] $client_package = $puppet::params::client_package,
   Boolean $agent = $puppet::params::agent,
   Boolean $remove_lock = $puppet::params::remove_lock,
+  Boolean $report = $puppet::params::report,
   Variant[String, Boolean] $client_certname = $puppet::params::client_certname,
   Optional[String] $puppetmaster = $puppet::params::puppetmaster,
   String $systemd_unit_name = $puppet::params::systemd_unit_name,

manifests/params.pp

@@ -15,6 +15,7 @@
   $splaylimit          = '1800'
   $runinterval         = 1800
   $runmode             = 'service'
+  $report              = true
 
   # Not defined here as the commands depend on module parameter "dir"
   $cron_cmd            = undef

spec/classes/puppet_agent_config_spec.rb

@@ -29,6 +29,7 @@
         end
 
         it { should contain_puppet__config__agent('certname').with(facts[:certname]) }
+        it { should contain_puppet__config__agent('report').with({'value'=>'true'}) }
       end
 
       context 'with runmode => cron', :unless => (facts[:osfamily] == 'Archlinux') do
@@ -79,6 +80,14 @@
 
         it { should_not contain_puppet__config__agent('certname') }
       end
+
+      context 'with report => false' do
+        let :pre_condition do
+          'class { "::puppet": report => false }'
+        end
+
+        it { should contain_puppet__config__agent('report').with({'value'=>'false'}) }
+      end
     end
   end
 end