From 430d5c14f49b88830b2ed95e40b75931646fc6e3 Mon Sep 17 00:00:00 2001
From: Tom Laermans <tom.laermans@powersource.cx>
Date: Fri, 26 Jun 2020 22:24:34 +0200
Subject: [PATCH] Allow setting service state through optional parameters

This commit allows you to disable the Samba service or ensure it's not running.
---
 manifests/server.pp | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/manifests/server.pp b/manifests/server.pp
index 2ef7b3d..662ac25 100644
--- a/manifests/server.pp
+++ b/manifests/server.pp
@@ -31,6 +31,9 @@
   # SELinux options
   $selinux_enable_home_dirs = false,
   $selinux_export_all_rw    = false,
+  # Default state for service
+  $service_enabled          = 'true',
+  $service_ensure           = 'running',
   # LDAP options
   $ldap_suffix              = undef,
   $ldap_url                 = undef,
@@ -45,8 +48,8 @@
   # Main package and service
   package { $::samba::params::package: ensure => 'installed' }
   service { $::samba::params::service:
-    ensure    => 'running',
-    enable    => true,
+    ensure    => $service_ensure,
+    enable    => $service_enabled,
     hasstatus => true,
     subscribe => File[$::samba::params::config_file],
   }