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

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

Thanks for your suggested MP, but I've taking that idea and just changed it to be a simple sleep before doing the tests. I've deployed it about a dozen times on a test openstack, and it worked every single time, and done a handful of tests locally which were fine too.

I went with the slightly larger sleep to accomodate any cloud that might be running a bit slower - we can always tweak it if we find its not long enough.

« Back to merge proposal