Code review comment for ~szeestraten/charm-nagios:bug/1864968

Revision history for this message
Xav Paice (xavpaice) wrote :

Many thanks for this patch.

I've made a few comments regarding the config options in config.yaml, around documentation. The change itself seems good, and has tested OK via a manual deployment test.

review: Needs Fixing

« Back to merge proposal