Merge ~vultaire/charm-nagios:allow-multiple-relation-data-sets-per-host into charm-nagios:master
Proposed by
Paul Goins
Status: | Merged |
---|---|
Approved by: | James Troup |
Approved revision: | 525739b24923babdde61f66b5caf292340c1181b |
Merged at revision: | 054394442a902a178477b9855a4e06d1aa974bea |
Proposed branch: | ~vultaire/charm-nagios:allow-multiple-relation-data-sets-per-host |
Merge into: | charm-nagios:master |
Prerequisite: | ~vultaire/charm-nagios:skip-dedupe-prefix-for-same-model |
Diff against target: |
35 lines (+8/-5) 1 file modified
hooks/monitors_relation_changed.py (+8/-5) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Garrett Neugent (community) | Approve | ||
Xav Paice (community) | Approve | ||
🤖 prod-jenkaas-bootstack | continuous-integration | Approve | |
BootStack Reviewers | Pending | ||
Review via email: mp+407788@code.launchpad.net |
Commit message
Keep writing config for newly-created config files
With the new logic, we try to write only the files we need to.
However, there may legitimately be cases where there's multiple sets
of data associated with the same host name (e.g. multiple nrpe units),
and at present, only the first set of such data will actually be
written.
This change tracks files which are being newly created, and will allow
multiple sets of relation data to be written to the same file, if
necessary.
To post a comment you must log in.
A CI job is currently in progress. A follow up comment will be added when it completes.