Add service_ensure option to allow control of services running or not

manifests/init.pp

@@ -21,11 +21,14 @@
 #   Boolean. Manage auto-creation of home directories on user login.
 #
 # [*enable_mkhomedir_flags*]
-#   Flags to use with authconfig to enable auto-creation of home directories. 
+#   Flags to use with authconfig to enable auto-creation of home directories.
 #
 # [*disable_mkhomedir_flags*]
 #   Flags to use with authconfig to disable auto-creation of home directories.
 #
+# [*service_ensure*]
+#   Ensure if services should be running/stopped
+#
 # === Examples
 #
 # class {'::sssd':
@@ -66,10 +69,12 @@
   $config                  = $sssd::params::config,
   $sssd_package            = $sssd::params::sssd_package,
   $sssd_service            = $sssd::params::sssd_service,
+  $sssd_ensure             = $sssd::params::sssd_ensure,
   $extra_packages          = $sssd::params::extra_packages,
   $config_file             = $sssd::params::config_file,
   $mkhomedir               = $sssd::params::mkhomedir,
   $manage_oddjobd          = $sssd::params::manage_oddjobd,
+  $service_ensure          = $sssd::params::service_ensure,
   $enable_mkhomedir_flags  = $sssd::params::enable_mkhomedir_flags,
   $disable_mkhomedir_flags = $sssd::params::disable_mkhomedir_flags,
 ) inherits sssd::params {
@@ -101,6 +106,8 @@
     $config
   )
 
+  validate_re($service_ensure, '^running|true|stopped|false$')
+
   anchor { 'sssd::begin': } ->
   class { '::sssd::install': } ->
   class { '::sssd::config': } ~>

manifests/params.pp

@@ -21,10 +21,11 @@
 
     'RedHat': {
 
-      $sssd_package = 'sssd'
-      $sssd_service = 'sssd'
-      $config_file  = '/etc/sssd/sssd.conf'
-      $mkhomedir    = true
+      $sssd_package   = 'sssd'
+      $sssd_service   = 'sssd'
+      $service_ensure = 'running'
+      $config_file    = '/etc/sssd/sssd.conf'
+      $mkhomedir      = true
 
       if versioncmp($::operatingsystemrelease, '6.0') < 0 {
         $extra_packages = [
@@ -45,6 +46,7 @@
 
       $sssd_package   = 'sssd'
       $sssd_service   = 'sssd'
+      $service_ensure = 'running'
       $config_file    = '/etc/sssd/sssd.conf'
       $mkhomedir      = true
       $extra_packages = [

manifests/service.pp

@@ -3,10 +3,10 @@
   $sssd_service   = $sssd::sssd_service,
   $mkhomedir      = $sssd::mkhomedir,
   $manage_oddjobd = $sssd::manage_oddjobd,
+  $service_ensure = $sssd::service_ensure,
 ) {
-
   service { $sssd_service:
-    ensure     => running,
+    ensure     => $service_ensure,
     enable     => true,
     hasstatus  => true,
     hasrestart => true,
@@ -15,7 +15,7 @@
   if $mkhomedir and $manage_oddjobd {
 
     service { 'oddjobd':
-      ensure     => running,
+      ensure     => $service_ensure,
       enable     => true,
       hasstatus  => true,
       hasrestart => true,

spec/classes/init_spec.rb

@@ -10,6 +10,13 @@
 
       it { is_expected.not_to contain_package('oddjob-mkhomedir') }
       it { is_expected.not_to contain_service('oddjobd') }
+
+      it { is_expected.to contain_service('sssd').with_ensure('running') }
+    end
+
+    context 'with service ensure stopped' do
+      let (:params) { { :service_ensure => 'stopped' } }
+      it { is_expected.to contain_service('sssd').with_ensure('stopped') }
     end
   end
   describe "on RedHat 6.6" do
@@ -22,6 +29,15 @@
 
       it { is_expected.to contain_package('oddjob-mkhomedir') }
       it { is_expected.to contain_service('oddjobd') }
+
+      it { is_expected.to contain_service('sssd').with_ensure('running') }
+      it { is_expected.to contain_service('oddjobd').with_ensure('running') }
+    end
+
+    context 'with service ensure stopped' do
+      let (:params) { { :service_ensure => 'stopped' } }
+      it { is_expected.to contain_service('sssd').with_ensure('stopped') }
+      it { is_expected.to contain_service('oddjobd').with_ensure('stopped') }
     end
   end
   describe "on RedHat 7.1" do
@@ -34,6 +50,15 @@
 
       it { is_expected.to contain_package('oddjob-mkhomedir') }
       it { is_expected.to contain_service('oddjobd') }
+
+      it { is_expected.to contain_service('sssd').with_ensure('running') }
+      it { is_expected.to contain_service('oddjobd').with_ensure('running') }
+    end
+
+    context 'with service ensure stopped' do
+      let (:params) { { :service_ensure => 'stopped' } }
+      it { is_expected.to contain_service('sssd').with_ensure('stopped') }
+      it { is_expected.to contain_service('oddjobd').with_ensure('stopped') }
     end
   end
   describe "on Debian 8.1" do
@@ -46,6 +71,13 @@
 
       it { is_expected.not_to contain_package('oddjob-mkhomedir') }
       it { is_expected.not_to contain_service('oddjobd') }
+
+      it { is_expected.to contain_service('sssd').with_ensure('running') }
+    end
+
+    context 'with service ensure stopped' do
+      let (:params) { { :service_ensure => 'stopped' } }
+      it { is_expected.to contain_service('sssd').with_ensure('stopped') }
     end
   end
 end