Merge lp:~thedac/charm-helpers/no-nrpe-from-charm-directory into lp:charm-helpers
Proposed by
David Ames
Status: | Merged |
---|---|
Merged at revision: | 103 |
Proposed branch: | lp:~thedac/charm-helpers/no-nrpe-from-charm-directory |
Merge into: | lp:charm-helpers |
Diff against target: |
13 lines (+0/-3) 1 file modified
charmhelpers/contrib/charmsupport/nrpe.py (+0/-3) |
To merge this branch: | bzr merge lp:~thedac/charm-helpers/no-nrpe-from-charm-directory |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tom Haddon | Approve | ||
Review via email:
|
Description of the change
NRPE checks should not be run from / or the charm directory.
Juju core permissions will not allow the nagios user to read from the charm directory so we should not imply it can be done in charm helpers
To post a comment you must log in.
I like it.