Add parameter to allow purging plugins, handlers, extensions and mutators

The default behaviour is not to purge, matching the existing behaviour.

Author: nhinds

manifests/init.pp

@@ -175,6 +175,11 @@
 #   Default: false
 #   Valid values: true, false
 #
+# [*purge_scripts*]
+#   Boolean.  If unused plugins, handlers, extensions and mutators should be removed from the system
+#   Default: false
+#   Valid values: true, false
+#
 # [*use_embedded_ruby*]
 #   Boolean.  If the embedded ruby should be used
 #   Default: false
@@ -234,14 +239,15 @@
   $plugins                  = [],
   $plugins_dir              = undef,
   $purge_config             = false,
+  $purge_scripts            = false,
   $use_embedded_ruby        = false,
   $rubyopt                  = '',
   $gem_path                 = '',
   $log_level                = 'info',
   $dashboard                = false,
 ){
 
-  validate_bool($client, $server, $api, $install_repo, $purge_config, $safe_mode, $manage_services)
+  validate_bool($client, $server, $api, $install_repo, $purge_config, $purge_scripts, $safe_mode, $manage_services)
 
   validate_re($repo, ['^main$', '^unstable$'], "Repo must be 'main' or 'unstable'.  Found: ${repo}")
   validate_re($version, ['^absent$', '^installed$', '^latest$', '^present$', '^[\d\.\-]+$'], "Invalid package version: ${version}")

manifests/package.pp

@@ -75,6 +75,8 @@
     mode    => '0555',
     owner   => 'sensu',
     group   => 'sensu',
+    recurse => true,
+    purge   => $sensu::purge_scripts,
     require => Package['sensu'],
   }
 
@@ -84,6 +86,8 @@
       mode    => '0555',
       owner   => 'sensu',
       group   => 'sensu',
+      recurse => true,
+      purge   => $sensu::purge_scripts,
       require => Package['sensu'],
     }
   }

spec/classes/sensu_package_spec.rb

@@ -14,6 +14,14 @@
           :purge  => false
         ) }
       end
+      [ '/etc/sensu/handlers', '/etc/sensu/extensions', '/etc/sensu/mutators',
+        '/etc/sensu/extensions/handlers', '/etc/sensu/plugins' ].each do |dir|
+        it { should contain_file(dir).with(
+          :ensure  => 'directory',
+          :recurse => true,
+          :purge   => false
+        ) }
+      end
       it { should contain_file('/etc/sensu/config.json').with_ensure('absent') }
       it { should contain_user('sensu') }
       it { should contain_group('sensu') }
@@ -153,6 +161,19 @@
       end
 
     end
+
+    context 'purge_scripts' do
+      let(:params) { { :purge_scripts => true } }
+
+      [ '/etc/sensu/handlers', '/etc/sensu/extensions', '/etc/sensu/mutators',
+        '/etc/sensu/extensions/handlers', '/etc/sensu/plugins' ].each do |dir|
+        it { should contain_file(dir).with(
+          :ensure  => 'directory',
+          :recurse => true,
+          :purge   => true
+        ) }
+      end
+    end
   end
 
 end