Skip to content

Commit fb04b13

Browse files
author
Andrei Neagu
committed
refactor
1 parent 6020209 commit fb04b13

File tree

1 file changed

+29
-37
lines changed
  • services/dynamic-scheduler/src/simcore_service_dynamic_scheduler/services/generic_scheduler

1 file changed

+29
-37
lines changed

services/dynamic-scheduler/src/simcore_service_dynamic_scheduler/services/generic_scheduler/_core.py

Lines changed: 29 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -366,49 +366,41 @@ async def _on_schedule_event(self, schedule_id: ScheduleId) -> None:
366366

367367
# 3) advancing operation in mode
368368
step_group_name = step_group.get_step_group_name(index=group_index)
369-
370-
def _log_as_mode(mode: str) -> None:
371-
_logger.debug(
372-
"%s step_group_name='%s' in operation_name='%s' for schedule_id='%s'",
373-
mode,
374-
step_group_name,
375-
operation_name,
376-
schedule_id,
377-
)
369+
base_message = f"{step_group_name=} in {operation_name=} for {schedule_id=}"
378370

379371
if step_group.repeat_steps is True and is_creating:
380-
_log_as_mode("REPEATING")
381-
await self._advance_as_repeating(
382-
schedule_data_proxy,
383-
schedule_id,
384-
operation_name,
385-
group_index,
386-
step_group,
387-
group_step_proxies,
388-
)
372+
with log_context(_logger, logging.DEBUG, f"REPEATING {base_message}"):
373+
await self._advance_as_repeating(
374+
schedule_data_proxy,
375+
schedule_id,
376+
operation_name,
377+
group_index,
378+
step_group,
379+
group_step_proxies,
380+
)
389381

390382
elif is_creating:
391-
_log_as_mode("CREATING")
392-
await self._advance_as_creating(
393-
steps_statuses,
394-
schedule_data_proxy,
395-
schedule_id,
396-
operation_name,
397-
group_index,
398-
step_group,
399-
operation,
400-
)
383+
with log_context(_logger, logging.DEBUG, f"CREATING {base_message}"):
384+
await self._advance_as_creating(
385+
steps_statuses,
386+
schedule_data_proxy,
387+
schedule_id,
388+
operation_name,
389+
group_index,
390+
step_group,
391+
operation,
392+
)
401393

402394
else:
403-
_log_as_mode("REVERTING")
404-
await self._advance_as_reverting(
405-
steps_statuses,
406-
schedule_data_proxy,
407-
schedule_id,
408-
operation_name,
409-
group_index,
410-
step_group,
411-
)
395+
with log_context(_logger, logging.DEBUG, f"REVERTING {base_message}"):
396+
await self._advance_as_reverting(
397+
steps_statuses,
398+
schedule_data_proxy,
399+
schedule_id,
400+
operation_name,
401+
group_index,
402+
step_group,
403+
)
412404

413405
async def _advance_as_repeating(
414406
self,

0 commit comments

Comments
 (0)