Giuseppe Lavagetto has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/344480 )
Change subject: role::memcached: convert to use profile::multidc::redis
......................................................................
role::memcached: convert to use profile::multidc::redis
Change-Id: I8868d4c597e275003fe1425abc98e7b344a37422
---
M hieradata/role/common/memcached.yaml
M modules/role/manifests/memcached.pp
2 files changed, 18 insertions(+), 58 deletions(-)
Approvals:
Giuseppe Lavagetto: Looks good to me, approved
jenkins-bot: Verified
diff --git a/hieradata/role/common/memcached.yaml
b/hieradata/role/common/memcached.yaml
index d90a586..13d7452 100644
--- a/hieradata/role/common/memcached.yaml
+++ b/hieradata/role/common/memcached.yaml
@@ -1,3 +1,18 @@
cluster: memcached
admin::groups:
- perf-roots
+profile::redis::multidc::category: sessions
+profile::redis::multidc::discovery: "appservers-rw"
+profile::redis::multidc::settings:
+ bind: '0.0.0.0'
+ auto_aof_rewrite_min_size: '512mb'
+ client_output_buffer_limit: 'slave 512mb 200mb 60'
+ dir: '/srv/redis'
+ maxmemory: '500Mb'
+ maxmemory_policy: 'volatile-lru'
+ maxmemory_samples: 5
+ no_appendfsync_on_rewrite: true
+ save: '300 100'
+ slave_read_only: false
+ stop_writes_on_bgsave_error: false
+profile::redis::multidc::aof: false
diff --git a/modules/role/manifests/memcached.pp
b/modules/role/manifests/memcached.pp
index dfaab0b..926525c 100644
--- a/modules/role/manifests/memcached.pp
+++ b/modules/role/manifests/memcached.pp
@@ -82,64 +82,6 @@
minute => fqdn_rand(59, 'memkeys'),
}
-
- $base_settings = {
- bind => '0.0.0.0',
- auto_aof_rewrite_min_size => '512mb',
- client_output_buffer_limit => 'slave 512mb 200mb 60',
- dir => '/srv/redis',
- dbfilename => "${::hostname}-6379.rdb",
- masterauth => $passwords::redis::main_password,
- maxmemory => '500Mb',
- maxmemory_policy => 'volatile-lru',
- maxmemory_samples => 5,
- no_appendfsync_on_rewrite => true,
- requirepass => $passwords::redis::main_password,
- save => '300 100',
- slave_read_only => false,
- stop_writes_on_bgsave_error => false,
- }
-
- $shards = {
- 'eqiad' => hiera('mediawiki::redis_servers::eqiad'),
- 'codfw' => hiera('mediawiki::redis_servers::codfw')
- }
-
- if os_version('Debian >= jessie') {
- class { 'redis::multidc::ipsec':
- shards => $shards
- }
- }
-
- class { 'redis::multidc::instances':
- shards => $shards,
- settings => $base_settings,
- map => {
- '6380' => {
- dbfilename => "${::hostname}-6380.rdb",
- }
- }
- }
-
-
- # Monitoring
-
- # Declare monitoring for all redis instances
- redis::monitoring::instance { $::redis::multidc::instances::instances:
- settings => $base_settings,
- map => $::redis::multidc::instances::replica_map,
- }
-
- # Firewall rules
- include ::ferm::ipsec_allow
-
- $redis_ports = join($::redis::multidc::instances::instances, ' ')
-
- ferm::service { 'redis_memcached_role':
- proto => 'tcp',
- port => inline_template('(<%= @redis_ports %>)'),
- }
-
ferm::service { 'memcached_memcached_role':
proto => 'tcp',
port => '11211',
@@ -149,4 +91,7 @@
content => template('role/memcached/rsyslog.conf.erb'),
priority => 40,
}
+
+ # Include redis for sessions
+ include profile::redis::multidc
}
--
To view, visit https://gerrit.wikimedia.org/r/344480
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8868d4c597e275003fe1425abc98e7b344a37422
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
