Dzahn has submitted this change and it was merged.

Change subject: Include base::firewall in the phabricator role
......................................................................


Include base::firewall in the phabricator role

Change-Id: I4b03900a1051dd4c4572b851f32dfc8d5c16e243
---
M manifests/role/phabricator.pp
M manifests/site.pp
2 files changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/manifests/role/phabricator.pp b/manifests/role/phabricator.pp
index 16e8f79..b4d5d2b 100644
--- a/manifests/role/phabricator.pp
+++ b/manifests/role/phabricator.pp
@@ -35,6 +35,7 @@
     include phabricator::monitoring
     include phabricator::mpm
     include lvs::realserver
+    include base::firewall
 
     $current_tag = 'release/2015-07-08/1'
     $domain = 'phabricator.wikimedia.org'
diff --git a/manifests/site.pp b/manifests/site.pp
index 6067828..90bee6c 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -1147,7 +1147,6 @@
     interface::add_ip6_mapped { 'main':
         interface => 'eth0',
     }
-    include base::firewall
     role phabricator::main
     include standard
     include ganglia

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4b03900a1051dd4c4572b851f32dfc8d5c16e243
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Chasemp <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to