♻️ docker compose reads compose spec file from stdin

[GitHK]

What do these changes do?

This is a drop-in change.

When the user's disk space is exhausted, the dynamic-sidecar, could no longer run any docker compose commands since each time a command was run, a temporary file was created.

Changed the docker compose to read the content of the file from stdin.

BONUS:

• ♻️ minor refactor to make commands more readable

Related issue/s

How to test

Dev Checklist

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

DevOps Checklist

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (146e3ec) 87.3% compared to head (59603a5) 67.0%.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #5231      +/-   ##
=========================================
- Coverage    87.3%   67.0%   -20.3%     
=========================================
  Files        1302     535     -767     
  Lines       53216   26875   -26341     
  Branches     1168     202     -966     
=========================================
- Hits        46481   18032   -28449     
- Misses       6485    8793    +2308     
+ Partials      250      50     -200     

Flag	Coverage Δ
integrationtests	63.7% <30.4%> (-1.4%)	⬇️
unittests	87.6% <100.0%> (+2.4%)	⬆️

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

Files	Coverage Δ
...rvice_dynamic_sidecar/core/docker_compose_utils.py	94.8% <100.0%> (-0.1%)	⬇️
.../src/simcore_service_dynamic_sidecar/core/utils.py	85.0% <100.0%> (-1.2%)	⬇️

... and 1070 files with indirect coverage changes

[sanderegg]

Looks great, but please adjust the names at least, they do not convey what is happening if I get that correctly.

[pcrespov]

ok. Added an extra suggestion

[sanderegg]

thanks, looking much better.

[sonarcloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud