Muehlenhoff has submitted this change and it was merged.

Change subject: Enable ferm on remaining API appservers in codfw
......................................................................


Enable ferm on remaining API appservers in codfw

Change-Id: Ic9aae66e4ec7b8fd4e4c4e0d64bd803340efc328
---
M manifests/site.pp
1 file changed, 3 insertions(+), 4 deletions(-)

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



diff --git a/manifests/site.pp b/manifests/site.pp
index 9320d45..5d6ae73 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -1966,10 +1966,7 @@
 #mw2050-2079 are api appservers
 node /^mw20[5-7][0-9]\.codfw\.wmnet$/ {
     role mediawiki::appserver::api
-
-    if $::hostname == 'mw2050' {
-        include base::firewall
-    }
+    include base::firewall
 }
 
 # ROW B codfw appservers: mw2080-mw2147
@@ -1996,6 +1993,7 @@
 #mw2120-2147 are api appservers
 node /^mw21([2-3][0-9]|4[0-7])\.codfw\.wmnet$/ {
     role mediawiki::appserver::api
+    include base::firewall
 }
 
 # ROW C codfw appservers: mw2148-mw2234
@@ -2020,6 +2018,7 @@
 #mw2200-2234 are api appservers
 node /^mw22([0-2][0-9]|3[0-4])\.codfw\.wmnet$/ {
     role mediawiki::appserver::api
+    include base::firewall
 }
 
 # Codfw ldap server, aka ldap-codfw

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic9aae66e4ec7b8fd4e4c4e0d64bd803340efc328
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <mmuhlenh...@wikimedia.org>
Gerrit-Reviewer: Muehlenhoff <mmuhlenh...@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