Muehlenhoff has submitted this change and it was merged.

Change subject: Don't enable ferm on logstash1004 yet
......................................................................


Don't enable ferm on logstash1004 yet

Needs additional changes

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

Approvals:
  Muehlenhoff: Verified; Looks good to me, approved



diff --git a/manifests/site.pp b/manifests/site.pp
index 1aaf3fd..9b8e5f2 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -1352,10 +1352,6 @@
 }
 node /^logstash100[4-6]\.eqiad\.wmnet$/ {
     role logstash::elasticsearch
-
-    if $::hostname == 'logstash1004' {
-        include base::firewall
-    }
 }
 
 node /lvs100[1-6]\.wikimedia\.org/ {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iac4d8ecccceff2c9524b3ce66ef9c9c9a2ef3cb4
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <mmuhlenh...@wikimedia.org>
Gerrit-Reviewer: Muehlenhoff <mmuhlenh...@wikimedia.org>

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

Reply via email to