Adjust logging to be more informative and clear

Signed-off-by: Mathias L. Baumann <[email protected]>

Author: Marenz

src/frequenz/dispatch/_actor_dispatcher.py

@@ -253,11 +253,13 @@ async def _stop_actor(self, stopping_dispatch: Dispatch, msg: str) -> None:
         identity = self._dispatch_identity(stopping_dispatch)
 
         if actor_and_channel := self._actors.pop(identity, None):
+            _logger.info("Stopping actor for dispatch type %r", stopping_dispatch.type)
             await actor_and_channel.actor.stop(msg)
             await actor_and_channel.channel.close()
         else:
             _logger.warning(
-                "Actor for dispatch type %r is not running", stopping_dispatch.type
+                "Actor for dispatch type %r is not running, ignoring stop request",
+                stopping_dispatch.type,
             )
 
     async def _run(self) -> None:

src/frequenz/dispatch/_merge_strategies.py

@@ -48,20 +48,37 @@ def filter(
             _logger.debug("Keeping start event %s", dispatch.id)
             return True
 
-        other_dispatches_running = any(
-            existing_dispatch.started_at(now)
+        running_dispatch_list = [
+            existing_dispatch
             for existing_dispatch in dispatches.values()
             if (
                 self.identity(existing_dispatch) == self.identity(dispatch)
                 and existing_dispatch.id != dispatch.id
             )
+        ]
+
+        other_dispatches_running = any(
+            running_dispatch.started_at(now)
+            for running_dispatch in running_dispatch_list
         )
 
         _logger.debug(
-            "stop event %s because other_dispatches_running=%s",
+            "%s stop event %s because other_dispatches_running=%s",
+            "Ignoring" if other_dispatches_running else "Allowing",
             dispatch.id,
             other_dispatches_running,
         )
+
+        if other_dispatches_running:
+            if _logger.isEnabledFor(logging.DEBUG):
+                _logger.debug(
+                    "Active other dispatches: %s",
+                    list(
+                        running_dispatch.id
+                        for running_dispatch in running_dispatch_list
+                    ),
+                )
+
         return not other_dispatches_running