🐛 Fix edge case when cancelling observation

[GitHK]

What do these changes do?

• ♻️ adds extra logs for the connection pool in the dynamic_sidecar_client
• 🐛 fixes an edge case where dynamic sidecars scheduler would end up in a loop calling _apply_observation_cycle and mark_service_for_removal. This was introduced when 🐛 observation cycle cancellation #3771 was added

Related issue/s

• improving dynamic-sidecar design osparc-issues#638

How to test

Checklist

• Unit tests for the changes exist

[codecov]

Codecov Report

Merging #3878 (9b47d87) into master (b88565c) will decrease coverage by 12.6%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #3878      +/-   ##
=========================================
- Coverage    85.0%   72.4%   -12.6%     
=========================================
  Files         933     383     -550     
  Lines       40015   19637   -20378     
  Branches      837     137     -700     
=========================================
- Hits        34042   14232   -19810     
+ Misses       5752    5356     -396     
+ Partials      221      49     -172     

Flag	Coverage Δ
integrationtests	66.6% <100.0%> (-0.2%)	⬇️
unittests	86.2% <100.0%> (+4.0%)	⬆️

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

Impacted Files	Coverage Δ
...tor_v2/modules/dynamic_sidecar/api_client/_base.py	98.6% <ø> (ø)
...ector_v2/modules/dynamic_sidecar/scheduler/_abc.py	100.0% <ø> (ø)
...dules/dynamic_sidecar/scheduler/_core/_observer.py	92.4% <ø> (-3.8%)	⬇️
...ules/dynamic_sidecar/scheduler/_core/_scheduler.py	78.2% <100.0%> (+0.1%)	⬆️
...ctor_v2/modules/dynamic_sidecar/scheduler/_task.py	94.1% <100.0%> (ø)
...r/src/simcore_service_webserver/storage_schemas.py	0.0% <0.0%> (-100.0%)	⬇️
...service_webserver/projects/_project_models_rest.py	0.0% <0.0%> (-100.0%)	⬇️
...es/web/server/src/simcore_service_webserver/log.py	0.0% <0.0%> (-94.8%)	⬇️
...erver/src/simcore_service_webserver/rest_models.py	0.0% <0.0%> (-92.7%)	⬇️
...simcore_service_webserver/director/director_api.py	0.0% <0.0%> (-76.0%)	⬇️
... and 696 more

[codeclimate]

Code Climate has analyzed commit 9b47d87 and detected 0 issues on this pull request.

View more on Code Climate.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.2% Duplication