Merge ~aluria/charm-openstack-service-checks/+git/charm-openstack-service-checks:bug/1825951 into ~canonical-bootstack/charm-openstack-service-checks:master

[Alvaro Uria]

Update the stored credentials on change

Closes-bug: #1825951

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

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

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

Unable to determine commit message from repository - please click "Set commit message" and enter the commit message manually.

[Alvaro Uria (aluria)]

MP approval depends on https://code.launchpad.net/~aluria/charm-openstack-service-checks/+git/charm-openstack-service-checks/+merge/366526

[Peter Sabaini (peter-sabaini)]

LGTM in general -- if poss. would suggest to add a test that demonstrates the issue and fix

review: Approve

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

Change has no commit message, setting status to needs review.

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

Change successfully merged at revision ec7a85429bdbf705811060b10412cc767d9a1852

diff --git a/reactive/openstack_service_checks.py b/reactive/openstack_service_checks.py
index 6fc43fd..63f5d62 100644
--- a/reactive/openstack_service_checks.py
+++ b/reactive/openstack_service_checks.py
@@ -105,6 +105,13 @@ def allow_keystone_store_overwrite():
     clear_flag('openstack-service-checks.stored-creds')
 
 
+@when('identity-credentials.available.updated')
+def update_keystone_store():
+    """when identity-service-relation-changed is triggered, update the stored credentials
+    """
+    allow_keystone_store_overwrite()
+
+
 def get_credentials():
     """Get credential info from either config or relation data