Remove start sync event from scheduler job.__call__

The scheduler job has a synchronization event (`start = asyncio.Event()`) with the purpose of executing job-group start,  and was passed as a parameter to `job.__call__`. It does not seems to be that necessary in the end as `asyncio.BoundedSemaphore` will handle job execution regardless. This commit removes this redundant event.

src/ert/scheduler/job.py

@@ -125,11 +125,8 @@ async def _submit_and_run_once(self, sem: asyncio.BoundedSemaphore) -> None:
                 timeout_task.cancel()
             sem.release()
 
-    async def run(
-        self, start: asyncio.Event, sem: asyncio.BoundedSemaphore, max_submit: int = 2
-    ) -> None:
+    async def run(self, sem: asyncio.BoundedSemaphore, max_submit: int = 2) -> None:
         self._requested_max_submit = max_submit
-        await start.wait()
         for attempt in range(max_submit):
             await self._submit_and_run_once(sem)
 

src/ert/scheduler/scheduler.py

@@ -273,15 +273,12 @@ async def execute(
             )
             scheduling_tasks.append(asyncio.create_task(self._update_avg_job_runtime()))
 
-        start = asyncio.Event()
         sem = asyncio.BoundedSemaphore(self._max_running or len(self._jobs))
         for iens, job in self._jobs.items():
             self._job_tasks[iens] = asyncio.create_task(
-                job.run(start, sem, self._max_submit), name=f"job-{iens}_task"
+                job.run(sem, self._max_submit), name=f"job-{iens}_task"
             )
 
-        start.set()
-
         try:
             await self._monitor_and_handle_tasks(scheduling_tasks)
         finally:

tests/unit_tests/scheduler/test_job.py

@@ -113,7 +113,7 @@ async def test_job_run_sends_expected_events(
     job.started.set()
 
     job_run_task = asyncio.create_task(
-        job.run(job.started, asyncio.Semaphore(), max_submit=max_submit)
+        job.run(asyncio.Semaphore(), max_submit=max_submit)
     )
 
     for attempt in range(max_submit):