Merge lp://qastaging/~tribaal/charms/trusty/rabbitmq-server/nagios-check-critical-to-unknown into lp://qastaging/~openstack-charmers-archive/charms/trusty/rabbitmq-server/next
Status: | Merged |
---|---|
Merged at revision: | 126 |
Proposed branch: | lp://qastaging/~tribaal/charms/trusty/rabbitmq-server/nagios-check-critical-to-unknown |
Merge into: | lp://qastaging/~openstack-charmers-archive/charms/trusty/rabbitmq-server/next |
Diff against target: |
12 lines (+1/-1) 1 file modified
scripts/check_rabbitmq_queues.py (+1/-1) |
To merge this branch: | bzr merge lp://qastaging/~tribaal/charms/trusty/rabbitmq-server/nagios-check-critical-to-unknown |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
OpenStack Charmers | Pending | ||
Review via email: mp+279232@code.qastaging.launchpad.net |
Description of the change
This simple branch changes the nagios probe for queue size from emitting a CRITICAL to emitting an UNKNOWN when the data file scanned exists but does not (yet) contain relevant information.
This change will slightly alter the nagios alert's behavior in the following way, *if* the nagios service is related before the file has been populated (behavior is unchanged if that connections takes more than ~3 minutes on average):
- Once the charm is connected, if the file is not populated yet, the nagios probe will switch to unknown instead of critical (triggering a soft alert instead of a hard one).
- If the file is populated fast enough, no behavior changes.
- Should the charm stop populationg the file at some point in the future, nagios will switch to a soft alert for the duration set by the admin, retrying until enough time elapses to switch to a hard alert.
This makes the behavior consistent with nagios documentation, and further makes mojo tests run reliably (since in the case of mojo tests, the nagios checks might be run before rabbitmq has had a chance to populate the data file).
charm_lint_check #14749 rabbitmq- server- next for tribaal mp279232
LINT OK: passed
Build: http:// 10.245. 162.77: 8080/job/ charm_lint_ check/14749/