esunar / test4

GNU General Public License v3.0
0 stars 0 forks source link

nrpe relations must use nrpe-specific endpoints if available #164

Open esunar opened 2 years ago

esunar commented 2 years ago

Although we alert if a required relation to nrpe is missing, we do not check the relation type.

For charms that expose a nrpe-external-master or local-monitors endpoint, we should alert if a relation toward nrpe is present, but only uses the juju-info endpoint.

For charms that do not expose such endpoints, a relation using juju-info should be deemed sufficient.


Imported from Launchpad using lp2gh.

esunar commented 2 years ago

(by ec0) I'd think that adding specific endpoint names to relation checking generally would be a good feature.