Merge lp://qastaging/~jacekn/charm-helpers/nrpe-hostname into lp://qastaging/charm-helpers
Proposed by
Jacek Nykis
Status: | Merged |
---|---|
Merged at revision: | 245 |
Proposed branch: | lp://qastaging/~jacekn/charm-helpers/nrpe-hostname |
Merge into: | lp://qastaging/charm-helpers |
Diff against target: |
42 lines (+15/-2) 2 files modified
charmhelpers/contrib/charmsupport/nrpe.py (+5/-2) tests/contrib/charmsupport/test_nrpe.py (+10/-0) |
To merge this branch: | bzr merge lp://qastaging/~jacekn/charm-helpers/nrpe-hostname |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jorge Niedbalski (community) | Approve | ||
Jacek Nykis (community) | Needs Resubmitting | ||
Tim Van Steenburgh | Approve | ||
Review via email: mp+218072@code.qastaging.launchpad.net |
Description of the change
Add optional hostname parameter to NRPE class. This is useful in MAAS environments where machines can have real hostnames in DNS.
To post a comment you must log in.
Hi Jacek, thanks for this improvement to charmhelpers!
+1 LGTM, tests all pass.
A Charmer will be along to merge these changes soon.