nagios-charmers team mailing list archive

[Ian Booth <ian.booth@xxxxxxxxxxxxx>]

The proposal to merge ~wallyworld/nagios-charm:cmr-aware into nagios-charm:master has been updated.

Description changed to:

Add support for using the new charm networking primitives so that the charm becomes cross model relations aware.

The main change is to use the new charm helpers network_get() API to ask for an ingress-address instead of a private-address via unit_get(). Some existing bash hooks were re-written in Python to make use of the functionality.

The charm deploys and functions just fine in the only previously supported scenario which was everything on one model. In the cross model case, the new network primitives ensure the correct IP address is used ti communicate with nrpe (not just the private IP address as was the case).

Requires https://git.launchpad.net/~wallyworld/nrpe-charm/commit/?id=393d5fb8ff4a878df5499bfb09a949e8db415897

To test live:

juju bootstrap
juju deploy ~/charms/nagios-charm
juju expose nagios
juju offer nagios:monitors
juju switch controller
juju deploy ubuntu
juju deploy ~/charms/nrpe-charm
juju add-relation ubuntu nrpe
juju add-relation nrpe:monitors default.nagios

Note: in the cross model scenario above, there's still an issue because juju's firewall functionality and charm helpers don't support ICMP, so the ping tests fail

For more details, see:
https://code.launchpad.net/~wallyworld/nagios-charm/+git/nagios-charm/+merge/332829
-- 
Your team Nagios Charm developers is requested to review the proposed merge of ~wallyworld/nagios-charm:cmr-aware into nagios-charm:master.