Code review comment for lp://qastaging/~freyes/charm-helpers/lp1670223

Revision history for this message
Felipe Reyes (freyes) wrote :

I chose to check the nagios use over the directory only because I was worried about the permissions, but now you mentioned, the cronjob runs as root just like the hook, so checking os.path.isdir('/var/lib/nagios') will do the trick as well and it's more explicit what it's needed.

I'll do the change.

« Back to merge proposal