Dzahn has submitted this change and it was merged.

Change subject: memcached: move ferm rules out of the module
......................................................................


memcached: move ferm rules out of the module

This allows more fine-grained configuration of rules for the respective
roles (in a followup commit).

The ms-fe* systems are already covered by the swift-memcached ferm
service.

The ferm rules presen in silver and californium as not needed, on
those hosts memcached only listens on localhost.

(The rule in the module will be removed in a subsequent commit after
double-checking that all rules are in place)

Change-Id: I37bff7340c8441f51a8e785d5cd135bab0f8f27c
---
M manifests/role/memcached.pp
1 file changed, 5 insertions(+), 0 deletions(-)

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



diff --git a/manifests/role/memcached.pp b/manifests/role/memcached.pp
index ad6291e..f694519 100644
--- a/manifests/role/memcached.pp
+++ b/manifests/role/memcached.pp
@@ -84,6 +84,11 @@
         port     => '6379',
     }
 
+    ferm::service {'memcached_memcached_role':
+        proto => 'tcp',
+        port  => '11211',
+    }
+
     rsyslog::conf { 'memkeys':
         content  => template('memcached/rsyslog.conf.erb'),
         priority => 40,

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I37bff7340c8441f51a8e785d5cd135bab0f8f27c
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: Yuvipanda <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to