Code review comment for lp:~thedac/charm-helpers/nrpe-usr-local-lib

Revision history for this message
Tom Haddon (mthaddon) wrote :

Looks good, approving, although we should review lines 5 and 6 of this diff, as I think it means we're allowing nagios plugins to be run from within the charm directory, which is a bad thing™ and may even be broken in juju-core.

review: Approve

« Back to merge proposal