← Back to team overview

nagios-charmers team mailing list archive

Re: [Merge] ~wallyworld/nagios-charm:cmr-aware into nagios-charm:master

 

Review: Needs Fixing

Hello,

I run a few tests on juju 2.0.2. I wanted to make sure that:
1. Changes work on slightly older juju
2. I can deploy either of the new charms while the other end is still on older version.

I deployed your nrpe charm but stock nagios and related them.

I found 2 problems:
1. nrpe ended up with hook errors:
hook failed: "monitors-relation-changed" for nrpe:monitors

I captures some logs here: https://pastebin.canonical.com/201725/

Eventually it sorted itself out but I think user experience is poor if the charm has transient hook errors.

2. Nagios end did not configure any checks

Could you please have a look?
-- 
https://code.launchpad.net/~wallyworld/nagios-charm/+git/nagios-charm/+merge/332829
Your team Nagios Charm developers is subscribed to branch nagios-charm:master.


References