🐛 Bugfix/instrumentation queue

[matusdrobuliak66]

What do these changes do?

• 🐛 Bugfix/instrumentation queue
  • divide exclusive and nonexclusive queues (they behave differently on _unsubscribe_from_rabbitmq

Related issue/s

How to test

Dev Checklist

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

DevOps Checklist

[codecov]

Codecov Report

Merging #5122 (bfeaf87) into master (7920def) will decrease coverage by 6.5%.
The diff coverage is 89.0%.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #5122      +/-   ##
=========================================
- Coverage    87.3%   80.9%    -6.5%     
=========================================
  Files        1273     528     -745     
  Lines       52317   26694   -25623     
  Branches     1133     198     -935     
=========================================
- Hits        45720   21605   -24115     
+ Misses       6357    5039    -1318     
+ Partials      240      50     -190     

Flag	Coverage Δ
integrationtests	63.5% <65.4%> (-1.4%)	⬇️
unittests	86.4% <89.0%> (+1.2%)	⬆️

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

Files	Coverage Δ
...server/notifications/_rabbitmq_consumers_common.py	100.0% <100.0%> (ø)
...tifications/_rabbitmq_exclusive_queue_consumers.py	64.2% <100.0%> (ø)
.../simcore_service_webserver/notifications/plugin.py	100.0% <100.0%> (ø)
...ications/_rabbitmq_nonexclusive_queue_consumers.py	79.3% <79.3%> (ø)

... and 871 files with indirect coverage changes

[sanderegg]

thank you for taking care of that one!

[pcrespov]

Left some suggestions to consider before merging. thx!

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[codeclimate]

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

View more on Code Climate.