Moved Exec['newaliases'] to services so it could be run after service restart

4d569c54 · Chris Denneen · Raphaël Pinson · 68380615 · 4d569c54 · 4d569c54
Commit 4d569c54 authored Mar 20, 2018 by Chris Denneen Committed by Raphaël Pinson May 09, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 7 deletions

manifests/files.pp manifests/files.pp +0 -7

manifests/service.pp manifests/service.pp +7 -0

No files found.
--- a/manifests/files.pp
+++ b/manifests/files.pp
@@ -52,13 +52,6 @@ class postfix::files {
    seltype => $postfix::params::aliasesseltype,
  }
-  # Aliases
-  exec { 'newaliases':
-    command     => '/usr/bin/newaliases',
-    refreshonly => true,
-    subscribe   => File['/etc/aliases'],
-  }
  # Config files
  if $mastercf_source {
    $mastercf_content = undef

--- a/manifests/service.pp
+++ b/manifests/service.pp
@@ -11,6 +11,13 @@ class postfix::service {
    hasstatus => true,
    restart   => $::postfix::params::restart_cmd,
  }
+  # Aliases
+  exec { 'newaliases':
+    command     => '/usr/bin/newaliases',
+    refreshonly => true,
+    subscribe   => File['/etc/aliases'],
+    require     => Service['postfix'],
+  }
  if $::osfamily == 'RedHat' {
    alternatives { 'mta':
      path    => '/usr/sbin/sendmail.postfix',