♻️ refactor stop service command ⚠️🚨

[GitHK]

⚠️ Dev-ops

This code has to be deployed with all dynamic-services stopped. The service stop API from webserver-> dynamic-scheduler changed in a backwards incompatible way.

What do these changes do?

Adds project_id and user_id to the stop command sent to the dynamic-scheduler. These changes are required to cover an edge case: enables stopping a service that is not tracked by the scheduler.

These changes are part of #5892

Related issue/s

• required by ✨ dynamic-scheduler pushes service state to the frontend (⚠️ devops) #5892

How to test

Dev-ops checklist

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.5%. Comparing base (cafbf96) to head (4a3c16b).
Report is 259 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #5924      +/-   ##
=========================================
+ Coverage    84.5%   87.5%    +2.9%     
=========================================
  Files          10    1223    +1213     
  Lines         214   53363   +53149     
  Branches       25    1072    +1047     
=========================================
+ Hits          181   46709   +46528     
- Misses         23    6419    +6396     
- Partials       10     235     +225     

Flag	Coverage Δ
integrationtests	63.5% <100.0%> (?)
unittests	85.5% <100.0%> (+0.9%)	⬆️

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

Files	Coverage Δ
.../api_schemas_dynamic_scheduler/dynamic_services.py	100.0% <100.0%> (ø)
...bitmq/rpc_interfaces/dynamic_scheduler/services.py	0.0% <ø> (ø)
...ore_service_dynamic_scheduler/api/rpc/_services.py	100.0% <100.0%> (ø)
...c_scheduler/services/director_v2/_public_client.py	92.0% <100.0%> (ø)
...mic_scheduler/services/director_v2/_thin_client.py	100.0% <ø> (ø)
...simcore_service_webserver/dynamic_scheduler/api.py	88.3% <ø> (ø)
...rvice_webserver/garbage_collector/_core_orphans.py	100.0% <100.0%> (ø)
...core_service_webserver/projects/_nodes_handlers.py	91.1% <100.0%> (ø)
...simcore_service_webserver/projects/projects_api.py	79.3% <ø> (ø)

... and 1201 files with indirect coverage changes

[sanderegg]

not sure I get why this change is needed. but I guess it will come with the next PR..

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.1% Duplication on New Code

See analysis details on SonarCloud

[matusdrobuliak66]

Thanks! 💯