Merge ~vultaire/charm-nrpe:host-container-check-rework into charm-nrpe:master
Proposed by
Paul Goins
Status: | Merged |
---|---|
Approved by: | James Troup |
Approved revision: | a01359b53f0c564adf766afdf054d1b969d83bc7 |
Merged at revision: | 989b71a53179c762a299f9085ce4d788d207b14e |
Proposed branch: | ~vultaire/charm-nrpe:host-container-check-rework |
Merge into: | charm-nrpe:master |
Diff against target: |
356 lines (+167/-74) 6 files modified
hooks/nrpe_helpers.py (+79/-74) tests/functional/tests/bundles/bionic.yaml (+6/-0) tests/functional/tests/bundles/focal.yaml (+6/-0) tests/functional/tests/bundles/xenial.yaml (+6/-0) tests/functional/tests/nrpe_tests.py (+66/-0) tests/functional/tests/tests.yaml (+4/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
🤖 prod-jenkaas-bootstack (community) | continuous-integration | Approve | |
BootStack Reviewers | Pending | ||
BootStack Reviewers | Pending | ||
Review via email: mp+407276@code.launchpad.net |
Commit message
Reworked some checks to only run on hosts
Some checks have limited to no meaning when run on containers. This
patch tries to change some of those checks so they're only enabled and
installed on host machines.
Do note that this does impact testing. Functional tests expect that
the main machines of the model are not containers, and a newly added
test for this functionality will fail if it is run on an LXD-based
Juju controller.
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.