fixes pylint

Author: pcrespov

services/web/server/src/simcore_service_webserver/garbage_collector/_tasks_core.py

@@ -19,6 +19,8 @@
 
 _logger = logging.getLogger(__name__)
 
+_GC_TASK_NAME = f"{__name__}._collect_garbage_periodically"
+
 
 def create_background_task_for_garbage_collection() -> CleanupContextFunc:
 
@@ -36,7 +38,9 @@ async def _collect_garbage_periodically() -> None:
             with log_context(_logger, logging.INFO, "Garbage collect cycle"):
                 await collect_garbage(app)
 
-        async for _ in setup_periodic_task(app, _collect_garbage_periodically):
+        async for _ in setup_periodic_task(
+            app, _collect_garbage_periodically, task_name=_GC_TASK_NAME
+        ):
             yield
 
     return _cleanup_ctx_fun

services/web/server/src/simcore_service_webserver/garbage_collector/_tasks_utils.py

@@ -11,9 +11,19 @@
 CleanupContextFunc = Callable[[web.Application], AsyncIterator[None]]
 
 
+def create_task_name(coro: Callable) -> str:
+    """
+    Returns a unique name for the task based on its module and function name.
+    This is useful for logging and debugging purposes.
+    """
+    return f"{coro.__module__}.{coro.__name__}"
+
+
 async def setup_periodic_task(
     app: web.Application,
     periodic_task_coro: Callable[[], Coroutine[None, None, None]],
+    *,
+    task_name: str | None = None,
 ) -> AsyncIterator[None]:
     """
     Generic setup and teardown for periodic background tasks.
@@ -23,15 +33,15 @@ async def setup_periodic_task(
         periodic_task_coro: The periodic task coroutine function (already decorated with @exclusive_periodic)
     """
     # setup
-    task_name = f"{periodic_task_coro.__module__}.{periodic_task_coro.__name__}"
+    task_name = task_name or create_task_name(periodic_task_coro)
 
     task = asyncio.create_task(
         periodic_task_coro(),
         name=task_name,
     )
 
     # prevents premature garbage collection of the task
-    app_task_key = f"tasks.{task_name}"
+    app_task_key = f"gc-tasks/{task_name}"
     if app_task_key in app:
         msg = f"Task {task_name} is already registered in the app state"
         raise ValueError(msg)