Merge ~cjohnston/charm-mongodb:fix-check-replicaset-name into charm-mongodb:master

Proposed by Chris Johnston
Status: Merged
Approved by: Xav Paice
Approved revision: 8154403590e3bc70a1b15e6aa678571c3195ab62
Merged at revision: 1faf038e3d7b4c099290dbdcaa744d71ed9b5bdb
Proposed branch: ~cjohnston/charm-mongodb:fix-check-replicaset-name
Merge into: charm-mongodb:master
Diff against target: 16 lines (+2/-1)
1 file modified
hooks/hooks.py (+2/-1)
Reviewer Review Type Date Requested Status
Xav Paice (community) Approve
Adam Dyess (community) Approve
BootStack Reviewers Pending
Review via email: mp+406413@code.launchpad.net

Commit message

Fix replica set nrpe check name

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Adam Dyess (addyess) wrote :

this change addresses the name of the nagios check LP#1914080
* check_check_replica_sets --> check_replica_sets_status

review: Approve
Revision history for this message
Xav Paice (xavpaice) wrote :

Currently fails lint, I've raised https://code.launchpad.net/~xavpaice/charm-mongodb/+git/charm-mongodb/+merge/406423 to work past that.

Change lgtm.

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 1faf038e3d7b4c099290dbdcaa744d71ed9b5bdb

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/hooks/hooks.py b/hooks/hooks.py
2index 254d11b..9a994b2 100755
3--- a/hooks/hooks.py
4+++ b/hooks/hooks.py
5@@ -1532,10 +1532,11 @@ def update_nrpe_config():
6 )
7
8 nrpe.add_check(
9- shortname="check_replica_sets",
10+ shortname="replica_sets_status",
11 description="replica set check {%s}" % current_unit,
12 check_cmd="check_replica_sets.py",
13 )
14+ nrpe.remove_check(shortname="check_replica_sets")
15
16 nrpe.write()
17

Subscribers

People subscribed via source and target branches

to all changes: