Cleanup on nodo::subsystem::monitor::master

diff --git a/manifests/subsystem/monitor/master.pp b/manifests/subsystem/monitor/master.pp
index d6f09b7437bd57130ae491487c9e4b017eb9f2d6..dc52391e2ccc35a801b8d1fecc2d30ef924b6de7 100644 (file)
--- a/manifests/subsystem/monitor/master.pp
+++ b/manifests/subsystem/monitor/master.pp
@@ -7,16 +7,10 @@ class nodo::subsystem::monitor::master {
     require => Package['nagios'],
   }
 
-  $main = hiera('nodo::role::master::main',        false)
-
-  if $main == true {
-    $ensure  = 'present'
-    $package = 'installed'
-    $service = 'running'
-  } else {
-    $ensure  = 'absent'
-    $package = 'purged'
-    $service = 'stopped'
+  $main   = hiera('nodo::role::master::main',        false)
+  $ensure = $main ? {
+    true    => 'present',
+    default => 'absent',
   }
 
   #
@@ -25,7 +19,7 @@ class nodo::subsystem::monitor::master {
   #     http://kuszelas.eu/~eyck/log/Howto/Munin.Rrdcached.html
   #
   package { 'rrdcached':
-    ensure => $package,
+    ensure => $ensure,
   }
 
   file { '/etc/default/rrdcached':
@@ -37,8 +31,10 @@ class nodo::subsystem::monitor::master {
     notify  => Service['rrdcached'],
   }
 
-  service { 'rrdcached':
-    ensure  => $service,
-    require => Package['rrdcached'],
+  if $main == true {
+    service { 'rrdcached':
+      ensure  => running,
+      require => Package['rrdcached'],
+    }
   }
 }