set active status on common handler

Author: paulomach

src/charm.py

@@ -302,7 +302,6 @@ def _on_database_created(self, _) -> None:
             return
         if self.unit.is_leader():
             self.app_peer_data["database-start"] = "true"
-        self.unit.status = ActiveStatus()
 
     def _on_endpoints_changed(self, _) -> None:
         """Handle the database endpoints changed event."""
@@ -320,6 +319,8 @@ def _on_peer_relation_changed(self, _) -> None:
                 # flag should be picked up just once
                 self.app_peer_data["database-start"] = "done"
 
+            self.unit.status = ActiveStatus()
+
     def _on_relation_broken(self, _) -> None:
         """Handle the database relation broken event."""
         self._stop_continuous_writes()

src/relations/legacy_mysql.py

@@ -62,9 +62,6 @@ def _on_relation_joined(self, event):
         # Set database-start to true to trigger common post relation tasks
         self.charm.app_peer_data["database-start"] = "true"
 
-        # set charm status
-        self.charm.unit.status = ActiveStatus()
-
     def _on_relation_broken(self, _):
         if not self.charm.unit.is_leader():
             # only leader handles the relation data