Merge pull request #42 from thehyve/nmbd

Also start nmbd on ubuntu (and other systems that split out smbd and nmbd service configuration files)

2 files changed, 14 insertions, 2 deletions

diff --git a/manifests/server/params.pp b/manifests/server/params.pp
index aa9674e..5f2fc3e 100644
--- a/manifests/server/params.pp
+++ b/manifests/server/params.pp
@@ -9,12 +9,14 @@ class samba::server::params {
     'Debian': {
       case $::operatingsystem{
         'Debian': { $service_name = 'samba' }
-        'Ubuntu': { $service_name = 'smbd' }
+        'Ubuntu': { $service_name = 'smbd'
+                    $nmbd_name = 'nmbd' }
         default: { $service_name = 'samba' }
       }
     }
     'Gentoo': { $service_name = 'samba' }
-    'Archlinux': { $service_name = 'smbd' }
+    'Archlinux': { $service_name = 'smbd'
+                   $nmbd_name = 'nmbd' }
 
     # Currently Gentoo has $::osfamily = "Linux". This should change in
     # Factor 1.7.0 <http://projects.puppetlabs.com/issues/17029>, so
diff --git a/manifests/server/service.pp b/manifests/server/service.pp
index a8797de..1a9fbe7 100644
--- a/manifests/server/service.pp
+++ b/manifests/server/service.pp
@@ -12,4 +12,14 @@ class samba::server::service (
     enable      => $enable,
     require     => Class['samba::server::config']
   }
+
+  if $nmbd_name != undef {
+    service { $nmbd_name :
+      ensure     => $ensure,
+      hasrestart => false,
+      enable     => $enable,
+      require    => Class['samba::server::config'],
+    }
+  }
+
 }