Merge ~hloeung/content-cache-charm:nagios-monitoring into content-cache-charm:master

Proposed by Haw Loeung
Status: Merged
Approved by: Haw Loeung
Approved revision: 323ddd0274630c286348461f353167aaebc12b4a
Merged at revision: 2521aada8e3ba781bcda54392c442c49c6c58972
Proposed branch: ~hloeung/content-cache-charm:nagios-monitoring
Merge into: content-cache-charm:master
Diff against target: 16 lines (+6/-0)
1 file modified
reactive/content_cache.py (+6/-0)
Reviewer Review Type Date Requested Status
Joel Sing (community) +1 Approve
Canonical IS Reviewers Pending
Review via email: mp+382557@code.launchpad.net

Commit message

Add monitoring for multiple HAProxy processes as a workaround to LP:1828496

To post a comment you must log in.
Revision history for this message
Joel Sing (jsing) wrote :

LGTM, thanks.

review: Approve (+1)
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
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 2521aada8e3ba781bcda54392c442c49c6c58972

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/reactive/content_cache.py b/reactive/content_cache.py
2index a20cdbb..af0376f 100644
3--- a/reactive/content_cache.py
4+++ b/reactive/content_cache.py
5@@ -428,6 +428,12 @@ def configure_nagios():
6 shortname=check_name, description='{} backend proxy check'.format(site), check_cmd=cmd
7 )
8
9+ # Ensure we only have two HAProxy processes around - LP:1828496
10+ check_name = 'check_haproxy_procs'
11+ description = 'HAProxy process count'
12+ cmd = '/usr/lib/nagios/plugins/check_procs -c2 -w2 -C haproxy'
13+ nrpe_setup.add_check(shortname=check_name, description=description, check_cmd=cmd)
14+
15 nrpe_setup.write()
16 reactive.set_flag('nagios-nrpe.configured')
17

Subscribers

People subscribed via source and target branches