Rush has submitted this change and it was merged.

Change subject: elasticsearch: fix fw canary regex
......................................................................


elasticsearch: fix fw canary regex

Don't quote things that aren't literal :)

Change-Id: I2249334cefcb4dfd88e53f648d93cde53e874da6
---
M manifests/site.pp
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Rush: Verified; Looks good to me, approved
  jenkins-bot: Verified
  Dzahn: Looks good to me, but someone else must approve



diff --git a/manifests/site.pp b/manifests/site.pp
index 9bfb19a..f8a8cff 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -855,7 +855,7 @@
 
     role elasticsearch::server
 
-    if $::hostname == 'elastic102[3-6]' {
+    if $::hostname =~ /^elastic102[3-6]$/ {
         include base::firewall
     }
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2249334cefcb4dfd88e53f648d93cde53e874da6
Gerrit-PatchSet: 4
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Rush <[email protected]>
Gerrit-Reviewer: DCausse <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Rush <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to