♻️ director-v2 api client enhancements

[GitHK]

What do these changes do?

Apparently the dynamic-schedueler is eating up all connections in the http client used to talk to the dy-sidecars. This causes the director to stop talking to them.

This tries to add more details about what is happening to the connection pool.

• using correct instance of the client (no longer creating a new one)
• better pool usage details

Related issue/s

How to test

Checklist

• Unit tests for the changes exist

[codecov]

Codecov Report

Merging #3839 (6ada13a) into master (4723546) will decrease coverage by 11.6%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #3839      +/-   ##
=========================================
- Coverage    84.0%   72.4%   -11.6%     
=========================================
  Files         892     381     -511     
  Lines       37746   19521   -18225     
  Branches      837     137     -700     
=========================================
- Hits        31721   14145   -17576     
+ Misses       5804    5327     -477     
+ Partials      221      49     -172     

Flag	Coverage Δ
integrationtests	66.7% <83.3%> (+5.0%)	⬆️
unittests	85.8% <100.0%> (+3.9%)	⬆️

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% <100.0%> (+1.3%)	⬆️
...tor_v2/modules/dynamic_sidecar/api_client/_thin.py	100.0% <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%)	⬇️
...rc/simcore_service_webserver/version_control_db.py	24.5% <0.0%> (-68.7%)	⬇️
...src/simcore_service_webserver/activity/handlers.py	29.3% <0.0%> (-63.8%)	⬇️
...ce_webserver/studies_dispatcher/_studies_access.py	22.7% <0.0%> (-63.7%)	⬇️
... and 665 more

[pcrespov]

pair reviewed.
pls remove duplicated client.
txh

[codeclimate]

Code Climate has analyzed commit 6ada13a and detected 0 issues on this pull request.

View more on Code Climate.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.2% Duplication