Merge pull request #2 from babiel/ape_service_reload

feat: Reload postfix config without terminating connections

Before this PR, puppet restarts the postfix service which leads to terminated connections for clients on config changes.

Only in rare situations a restart of postfix is required to make changes effectiv: http://www.postfix.org/postconf.5.html

Changing the restart command to reload is also non-breaking as puppet starts a stopped service on notification by other/depending resources.

The default restart command is using systemd as it's the default service management on modern systems.
This may be has to be reworked when rebasing babiel/puppet-postfix on bodgit/puppet-postfix to bring this improvement to upstream, but at the moment both projects diverge a lot which will require additional work in the future.

Author: pecharmin

manifests/params.pp

@@ -4,6 +4,7 @@
   $conf_dir     = '/etc/postfix'
   $package_name = 'postfix'
   $service_name = 'postfix'
+  $service_reload_command = '/usr/bin/systemctl reload postfix.service'
 
   case $::osfamily {
     'RedHat': {

manifests/service.pp

@@ -6,5 +6,6 @@
     enable     => true,
     hasstatus  => true,
     hasrestart => true,
+    restart    => $::postfix::service_reload_command,
   }
 }