Code review comment for lp:~brad-marshall/charms/trusty/nagios/add-extra-config-options

Revision history for this message
Brad Marshall (brad-marshall) wrote :

I've added the missing sudo to the apt-get update. I'm not sure how to handle the race condition, all the try block does is checks for a file existence. Maybe I should add extra to the timeout on the sentry wait?

« Back to merge proposal