♻️Sim4Life computational jobs are not parented correctly

[sanderegg]

What do these changes do?

Issue: the project metadata are not set correctly in the comp_runs table when sim4life runs a job

Observations:

• Tried to reproduce the issue using a Jupyterlab and running the osparc client to no avail, it works there properly,
• Tried to add the exact same metadata and it still works.

Actions:

• added a debug log to see if the call in the director-v2 is doing some weird behavior.
• improved testing in dv-2 regarding metadata

Related issue/s

• relates to Parent project and node are wrong in resource_tracker_service_runs table #6530

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 68.1%. Comparing base (cafbf96) to head (2b3df56).
Report is 644 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #6542      +/-   ##
=========================================
- Coverage    84.5%   68.1%   -16.5%     
=========================================
  Files          10     604     +594     
  Lines         214   30656   +30442     
  Branches       25     265     +240     
=========================================
+ Hits          181   20895   +20714     
- Misses         23    9700    +9677     
- Partials       10      61      +51     

Flag	Coverage Δ
integrationtests	64.7% <100.0%> (?)
unittests	84.0% <100.0%> (-0.6%)	⬇️

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

Files with missing lines	Coverage Δ
...ore_service_director_v2/api/routes/computations.py	84.0% <100.0%> (ø)
...e_director_v2/modules/db/repositories/comp_runs.py	94.9% <100.0%> (ø)

... and 612 files with indirect coverage changes

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[mguidon]

Just fyi: s4l in prod is still using custom meta data fields for this. Not the headers.

[pcrespov]

IF I understand this correctly, you still did not manage to find the cause for the bug and this is just adding some loggers to get more info in the master deployment.

[sanderegg]

IF I understand this correctly, you still did not manage to find the cause for the bug and this is just adding some loggers to get more info in the master deployment.

yes