Giuseppe Lavagetto has submitted this change and it was merged.
Change subject: role::jobqueue_redis: add monitoring of the redis instances
......................................................................
role::jobqueue_redis: add monitoring of the redis instances
Change-Id: I7b3539d38d73a6f2c20a1f815ba6d69cb1fe631b
---
M manifests/role/jobqueue_redis.pp
1 file changed, 6 insertions(+), 0 deletions(-)
Approvals:
Giuseppe Lavagetto: Verified; Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/role/jobqueue_redis.pp b/manifests/role/jobqueue_redis.pp
index 1538787..442902e 100644
--- a/manifests/role/jobqueue_redis.pp
+++ b/manifests/role/jobqueue_redis.pp
@@ -14,6 +14,10 @@
# The instances are the same we'd find on its master
$instances = redis_get_instances($slaveof, $shards)
mediawiki::jobqueue_redis { $instances: slaveof => $slaveof}
+ # Monitoring
+ redis::monitoring::instance { $instances:
+ settings => {slaveof => $slaveof}
+ }
} else {
# Local master
# Encrypt the replication
@@ -26,6 +30,8 @@
# find out the replication topology
$slaves_map = redis_add_replica({}, $ip, $shards, $::mw_primary)
mediawiki::jobqueue_redis {$instances: map => $slaves_map}
+ # Monitoring
+ redis::monitoring::instance { $instances: map => $slaves_map}
}
$uris = apply_format("localhost:%s/${password}", $instances)
--
To view, visit https://gerrit.wikimedia.org/r/282950
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7b3539d38d73a6f2c20a1f815ba6d69cb1fe631b
Gerrit-PatchSet: 4
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
