fix weird issue

Author: sanderegg

services/dynamic-scheduler/src/simcore_service_dynamic_scheduler/services/status_monitor/_monitor.py

@@ -10,7 +10,6 @@
 from pydantic import NonNegativeFloat, NonNegativeInt
 from servicelib.async_utils import cancel_wait_task
 from servicelib.background_task_utils import exclusive_periodic
-from servicelib.redis._client import RedisClientSDK
 from servicelib.utils import limited_gather
 from settings_library.redis import RedisDatabase
 
@@ -62,10 +61,6 @@ def _can_be_removed(model: TrackedServiceModel) -> bool:
     return False
 
 
-def _get_redis_client_from_monitor(monitor: "Monitor") -> RedisClientSDK:
-    return get_redis_client(monitor.app, RedisDatabase.LOCKS)
-
-
 class Monitor:
     def __init__(self, app: FastAPI, status_worker_interval: timedelta) -> None:
         self.app = app
@@ -75,11 +70,6 @@ def __init__(self, app: FastAPI, status_worker_interval: timedelta) -> None:
     def status_worker_interval_seconds(self) -> NonNegativeFloat:
         return self.status_worker_interval.total_seconds()
 
-    @exclusive_periodic(
-        _get_redis_client_from_monitor,
-        task_interval=_INTERVAL_BETWEEN_CHECKS,
-        retry_after=_INTERVAL_BETWEEN_CHECKS,
-    )
     async def _worker_check_services_require_status_update(self) -> None:
         """
         Check if any service requires it's status to be polled.
@@ -144,8 +134,16 @@ async def _worker_check_services_require_status_update(self) -> None:
         )
 
     async def setup(self) -> None:
+        @exclusive_periodic(
+            get_redis_client(self.app, RedisDatabase.LOCKS),
+            task_interval=_INTERVAL_BETWEEN_CHECKS,
+            retry_after=_INTERVAL_BETWEEN_CHECKS,
+        )
+        async def _periodic_check_services_require_status_update() -> None:
+            await self._worker_check_services_require_status_update()
+
         self.app.state.status_monitor_background_task = asyncio.create_task(
-            self._worker_check_services_require_status_update(),
+            _periodic_check_services_require_status_update(),
             name="periodic_service_status_update",
         )