Merge ~xavpaice/charm-openstack-service-checks/+git/charm-openstack-service-checks:reconfigure-on-configchanged into ~canonical-bootstack/charm-openstack-service-checks:master

[Xav Paice]

Add config-changed

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[Andrea Ieri (aieri)]

This seems to be overlapping with:
https://code.launchpad.net/~aluria/charm-openstack-service-checks/+git/charm-openstack-service-checks/+merge/366581
https://code.launchpad.net/~aluria/charm-openstack-service-checks/+git/charm-openstack-service-checks/+merge/366526

It might be better to merge those two first, and see if this one is still needed.

[Xav Paice (xavpaice)]

We need to clear the .configured flag in order to trigger regeneration of the checks, and nothing in the other two changes does that, so I think that they don't actually overlap.

[Wouter van Bommel (woutervb)]

Makes sence

review: Approve

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision 869d5d5432e3baa5f9baa54ee716ecc1aff8e3ef

diff --git a/reactive/openstack_service_checks.py b/reactive/openstack_service_checks.py
index c122bf1..6a0d842 100644
--- a/reactive/openstack_service_checks.py
+++ b/reactive/openstack_service_checks.py
@@ -31,6 +31,11 @@ CERT_FILE = '/usr/local/share/ca-certificates/openstack-service-checks.crt'
 helper = OSCHelper()
 
 
+@when('config.changed')
+def config_changed():
+    clear_flag('openstack-service-checks.configured')
+
+
 @when_not('openstack-service-checks.installed')
 @when('nrpe-external-master.available')
 def install_openstack_service_checks():