Merge pull request #14 from powerhome/bugfix/service_manage

Fix service notifies for config changes
markt-de · Dec 19, 2016 · 7eeca04 · 7eeca04
2 parents 969f4c3 + 96daf61
commit 7eeca04
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/manifests/config.pp b/manifests/config.pp
@@ -82,7 +82,6 @@
     File {
       ensure  => present,
       require => Class['::proftpd::install'],
-      notify  => Service[$::proftpd::service_name],
     }
 
     file {
@@ -108,14 +107,16 @@
         validate_cmd => "${::proftpd::prefix_bin}/proftpd -t -c %",
         owner        => $::proftpd::config_user,
         group        => $::proftpd::config_group,
-        require      => $config_require;
+        require      => $config_require,
+        notify       => Class[::proftpd::service];
     }
 
     concat { $modules_config:
       owner  => $::proftpd::config_user,
       group  => $::proftpd::config_group,
       # modules may be required for validate_cmd to succeed
       before => File[$::proftpd::config],
+      notify => Class[::proftpd::service],
     }
 
     concat::fragment { 'proftp_modules_header':