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=ce33aa27a612ca2c4ade6f78e0a8f0492cb7880a

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



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.