Merge ~guoqiao/charm-nrpe:add-update-status-hook into charm-nrpe:master
Proposed by
Joe Guo
Status: | Merged |
---|---|
Approved by: | Xav Paice |
Approved revision: | 5342ff7d72b24d8eb79e4febce1f8f9685345fc9 |
Merged at revision: | dcdef1bd268cba8a2efb3fdf50ef4e41b03320e8 |
Proposed branch: | ~guoqiao/charm-nrpe:add-update-status-hook |
Merge into: | charm-nrpe:master |
Diff against target: |
71 lines (+39/-11) 2 files modified
hooks/services.py (+16/-11) hooks/update-status (+23/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Xav Paice (community) | Approve | ||
🤖 prod-jenkaas-bootstack | continuous-integration | Needs Fixing | |
Linda Guo (community) | Approve | ||
BootStack Reviewers | Pending | ||
Review via email: mp+409732@code.staging.launchpad.net |
Commit message
Add missing update-status hook
Currently in nrpe charm, the update-status hook is missing. We keep seeing this message in juju log:
unit-nrpe-0: 20:47:06 INFO juju.worker.
This patch added a separate python script for this hook, to check nagios-nrpe-server status and use status-set hook tool to update status.
The status message is kept identical to before.
To post a comment you must log in.
There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.
This merge proposal is being monitored by mergebot. Change the status to Approved to merge.