openHPI / poseidon

Scalable task execution orchestrator for CodeOcean
MIT License
8 stars 1 forks source link

Fix Nomad runner recovery #614

Closed mpass99 closed 3 months ago

mpass99 commented 3 months ago

Closes #598

that led to missing runners when Nomad created runners faster than the recovery process takes.

By starting the Event Stream not after, but with the start of the runner recovery, Poseidon can register the runners started in the recovery process.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 76.92%. Comparing base (342b937) to head (01d6cb8). Report is 2 commits behind head on main.

Files Patch % Lines
cmd/poseidon/main.go 83.33% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #614 +/- ## ======================================= Coverage 76.92% 76.92% ======================================= Files 41 41 Lines 3497 3498 +1 ======================================= + Hits 2690 2691 +1 Misses 592 592 Partials 215 215 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.