[DPE-5711] Add warnings to destructive actions (#188)

canonical · Nov 28, 2024 · 950a9ea · 950a9ea
1 parent 0f42207
commit 950a9ea
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/machine_charm.py b/src/machine_charm.py
@@ -197,14 +197,15 @@ def _on_force_upgrade_action(self, event: ops.ActionEvent) -> None:
             logger.debug(f"Force upgrade event failed: {message}")
             event.fail(message)
             return
-        logger.debug("Forcing upgrade")
+
+        logger.warning("Forcing upgrade")
         event.log(f"Forcefully upgrading {self.unit.name}")
         self._upgrade.upgrade_unit(
             event=event, workload_=self.get_workload(event=None), tls=self._tls_certificate_saved
         )
         self.reconcile()
         event.set_results({"result": f"Forcefully upgraded {self.unit.name}"})
-        logger.debug("Forced upgrade")
+        logger.warning("Forced upgrade")
 
 
 if __name__ == "__main__":

diff --git a/src/workload.py b/src/workload.py
@@ -145,14 +145,14 @@ def _disable_tls(self) -> None:
 
     def _disable_router(self) -> None:
         """Disable router and clean up corresponding router files."""
-        logger.debug("Disabling MySQL Router service")
+        logger.info("Disabling MySQL Router service")
         self._container.update_mysql_router_service(enabled=False)
         self._logrotate.disable()
         self._container.router_config_directory.rmtree()
         self._container.router_config_directory.mkdir()
         self._router_data_directory.rmtree()
         self._router_data_directory.mkdir()
-        logger.debug("Disabled MySQL Router service")
+        logger.info("Disabled MySQL Router service")
 
     def reconcile(
         self,