🐛 Fix log streaming issues

[bisgaard-itis]

What do these changes do?

• Refactor log streaming. Now there is a single wrapped RabbitMQ client (called the LogDistributor) which subscribes to the log exchange in the RabbitMQ services and distributes the logs from different jobs to the corresponding LogStreamers
• When a request to the logstream endpoint is made a LogStreamer is constructed and it registers itself with the LogDistributor.
• Add unit tests for having multiple users streaming at the same time and requesting the logstream multiple times.
• I also allow an arbitrarily large queue in the LogStreamer as otherwise logs are lost when streaming. Seemingly that was the cause of Logs disappearing on their way from computational sidecar to api-server #5096

Related issue/s

• resolves Minor issues with logstreaming #5081
• resolves Logs disappearing on their way from computational sidecar to api-server #5096

How to test

Dev Checklist

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

DevOps Checklist

[codecov]

Codecov Report

Merging #5104 (dba4b44) into master (70efe70) will decrease coverage by 21.0%.
The diff coverage is 90.2%.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #5104      +/-   ##
=========================================
- Coverage    87.3%   66.4%   -21.0%     
=========================================
  Files        1267     596     -671     
  Lines       52079   29227   -22852     
  Branches     1129     198     -931     
=========================================
- Hits        45507   19424   -26083     
- Misses       6332    9753    +3421     
+ Partials      240      50     -190     

Flag	Coverage Δ
integrationtests	64.8% <ø> (-0.1%)	⬇️
unittests	83.4% <90.2%> (-1.8%)	⬇️

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

Files	Coverage Δ
...re_service_api_server/api/dependencies/rabbitmq.py	81.8% <100.0%> (-14.3%)	⬇️
...vice_api_server/api/routes/solvers_jobs_getters.py	81.2% <100.0%> (+0.2%)	⬆️
...src/simcore_service_api_server/core/application.py	97.3% <100.0%> (+0.1%)	⬆️
...c/simcore_service_api_server/models/basic_types.py	100.0% <100.0%> (ø)
.../simcore_service_api_server/models/schemas/jobs.py	96.5% <100.0%> (+<0.1%)	⬆️
...rc/simcore_service_api_server/services/rabbitmq.py	100.0% <100.0%> (ø)
...mcore_service_api_server/services/log_streaming.py	93.8% <93.8%> (ø)
...ervice_api_server/api/errors/log_handling_error.py	46.1% <46.1%> (ø)

... and 999 files with indirect coverage changes

[pcrespov]

Great job. Left some comments for your consideration. We should also discuss about error handling at some point. We need to find a common policy for the api-server.

[mrnicegyu11]

supernice, thanks, I left some minor comments, good luck with this!

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

No Coverage information
0.0% Duplication

[codeclimate]

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

View more on Code Climate.

[pcrespov]

🎉 great work!

[sanderegg]

just a few questions around. otherwise great. thanks!