Fix: enable/disable service depending on the ensure parameter

diff --git a/manifests/init.pp b/manifests/init.pp
index 2af37732418e464ac71405e1e8f5374fda754a97..0a8583871ef2348048a8c3db20004f7f42e83ad3 100644 (file)
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -1,18 +1,18 @@
 # This module is distributed under the GNU Affero General Public License:
 # This module is distributed under the GNU Affero General Public License:

-# 
+#

 # Apcupsd module for puppet
 # Copyright (C) 2010 Sarava Group
 # Apcupsd module for puppet
 # Copyright (C) 2010 Sarava Group

-# 
+#

 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU Affero General Public License as
 # published by the Free Software Foundation, either version 3 of the
 # License, or any later version.
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU Affero General Public License as
 # published by the Free Software Foundation, either version 3 of the
 # License, or any later version.

-# 
+#

 # This program is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU Affero General Public License for more details.
 # This program is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU Affero General Public License for more details.

-# 
+#

 # You should have received a copy of the GNU Affero General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 # You should have received a copy of the GNU Affero General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 


@@ -81,7 +81,10 @@ class apcupsd(
   }
 
   service { "apcupsd":
   }
 
   service { "apcupsd":

-    enable     => true,
+    enable => $ensure ? {
+      'present' => true,
+      'absent'  => false,
+    },

     ensure     => $ups_state,
     hasrestart => true,
     require    => [ File["/etc/apcupsd/apcupsd.conf"], Package["apcupsd"] ],
     ensure     => $ups_state,
     hasrestart => true,
     require    => [ File["/etc/apcupsd/apcupsd.conf"], Package["apcupsd"] ],