From be53a7dd5238e20c5dc37d25400a14e52589d6e7 Mon Sep 17 00:00:00 2001 From: Wyatt Rees Date: Wed, 30 Oct 2024 12:41:37 +0100 Subject: [PATCH] Use stored state to track initialization of MAAS --- maas-region/src/charm.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/maas-region/src/charm.py b/maas-region/src/charm.py index 31e0bd1..651a2a1 100755 --- a/maas-region/src/charm.py +++ b/maas-region/src/charm.py @@ -118,7 +118,7 @@ def __init__(self, *args): # Charm configuration self.framework.observe(self.on.config_changed, self._on_config_changed) - self.maas_initialized_once = False + self._stored.set_default(maas_initialized_once=False) @property def peers(self) -> Union[ops.Relation, None]: @@ -237,9 +237,9 @@ def _initialize_maas(self) -> bool: MaasHelper.setup_region( self.maas_api_url, self.connection_string, self.get_operational_mode() ) - if not self.maas_initialized_once: + if not self._stored.maas_initialized_once: self._update_tls_config() - self.maas_initialized_once = True + self._stored.maas_initialized_once = True return True except subprocess.CalledProcessError: return False @@ -470,7 +470,7 @@ def _on_config_changed(self, event: ops.ConfigChangedEvent): "Both ssl_cert_content and ssl_key_content must be defined when using tls_mode=passthrough" ) self._update_ha_proxy() - if self.maas_initialized_once: + if self._stored.maas_initialized_once: self._update_tls_config()