Log the full actor name when restarting

The actor `name` is really just a unique ID, and lacks the class name.
For logging is best to use the `str`, which includes both the class name
and the ID.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

src/frequenz/sdk/actor/_actor.py

@@ -99,7 +99,7 @@ async def _run_loop(self) -> None:
                 limit_str = f"({n_restarts}/{limit_str})"
                 if self._restart_limit is None or n_restarts < self._restart_limit:
                     n_restarts += 1
-                    _logger.info("Actor %s: Restarting %s...", self._name, limit_str)
+                    _logger.info("Actor %s: Restarting %s...", self, limit_str)
                     continue
                 _logger.info(
                     "Actor %s: Maximum restarts attempted %s, bailing out...",

tests/actor/test_actor.py

@@ -245,7 +245,7 @@ async def test_restart_on_unhandled_exception(
                 ),
                 (
                     *ACTOR_INFO,
-                    f"Actor test: Restarting ({i}/{restart_limit})...",
+                    f"Actor RaiseExceptionActor[test]: Restarting ({i}/{restart_limit})...",
                 ),
                 (
                     *ACTOR_INFO,