🐛Fix infinite Waiting for cluster when dask-scheduler is restarted

[sanderegg]

What do these changes do?

This PR fixes #5237 by:

• NOT swallowing a raised exception
• a bit of refactoring
  I will intentionally not undergo heavy refactoring now

Related issue/s

fixes #5237

How to test

Dev Checklist

• No ENV changes or I properly updated ENV (read the instruction)

DevOps Checklist

[codecov]

Codecov Report

Attention: 19 lines in your changes are missing coverage. Please review.

Comparison is base (42d7630) 86.1% compared to head (fbba568) 68.7%.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #5252      +/-   ##
=========================================
- Coverage    86.1%   68.7%   -17.4%     
=========================================
  Files         864     535     -329     
  Lines       38093   26897   -11196     
  Branches      347     202     -145     
=========================================
- Hits        32814   18491   -14323     
- Misses       5202    8356    +3154     
+ Partials       77      50      -27     

Flag	Coverage Δ
integrationtests	65.0% <60.0%> (+10.8%)	⬆️
unittests	84.6% <70.7%> (-0.5%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...mcore_service_director_v2/models/dask_subsystem.py	100.0% <100.0%> (ø)
...rector_v2/modules/comp_scheduler/dask_scheduler.py	89.7% <100.0%> (-0.1%)	⬇️
...tor_v2/modules/db/repositories/comp_tasks/_core.py	98.0% <100.0%> (ø)
...r-v2/src/simcore_service_director_v2/utils/dask.py	89.8% <100.0%> (+<0.1%)	⬆️
...simcore_service_director_v2/modules/dask_client.py	92.5% <84.8%> (-0.7%)	⬇️
...rector_v2/modules/comp_scheduler/base_scheduler.py	86.2% <41.6%> (-0.3%)	⬇️

... and 592 files with indirect coverage changes

[pcrespov]

Excellent. Did a pair review and concluded some interesting ideas:

[GitHK]

👍

[sonarcloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud