Faidon Liambotis has submitted this change and it was merged.

Change subject: mail: add network::constants include to role class
......................................................................


mail: add network::constants include to role class

role::mail::mx was missing it. Remove it from exim::roled and add it
there instead, for both role::mail::mx and role::mail::lists.

Also cleanup exim4::ganglia includes while at it.

Change-Id: If94178a64fb206091a9467b69ac78da35e3ba253
---
M manifests/mail.pp
M manifests/role/mail.pp
2 files changed, 6 insertions(+), 4 deletions(-)

Approvals:
  Faidon Liambotis: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/manifests/mail.pp b/manifests/mail.pp
index 62c9417..9f17477 100644
--- a/manifests/mail.pp
+++ b/manifests/mail.pp
@@ -54,8 +54,8 @@
 ) {
 
         include exim::smtp
-        include network::constants
         include privateexim::listserve
+        include exim4::ganglia
 
         class { 'exim4':
             variant => 'heavy',
diff --git a/manifests/role/mail.pp b/manifests/role/mail.pp
index 6a6e17c..4fbaa05 100644
--- a/manifests/role/mail.pp
+++ b/manifests/role/mail.pp
@@ -6,8 +6,8 @@
 }
 
 class role::mail::mx {
+    include network::constants
     include privateexim::aliases::private
-    include exim4::ganglia
 
     mailalias { 'root':
         recipient => 'r...@wikimedia.org',
@@ -51,15 +51,17 @@
         enabled => true,
     }
 
-    # FIXME: the rest is unpuppetized so far
-
     monitor_service { 'smtp':
         description   => 'Exim SMTP',
         check_command => 'check_smtp',
     }
+
+    # FIXME: the rest is unpuppetized so far
 }
 
 class role::mail::lists {
+    include network::constants
+
     # FIXME: needs to be split to lists/secondarymx
 
     system::role { 'role::mail::lists':

-- 
To view, visit https://gerrit.wikimedia.org/r/141415
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If94178a64fb206091a9467b69ac78da35e3ba253
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <fai...@wikimedia.org>
Gerrit-Reviewer: Faidon Liambotis <fai...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to