ITISFoundation / osparc-simcore

🐼 osparc-simcore simulation framework
https://osparc.io
MIT License
44 stars 26 forks source link

👽️ Add endpoint for getting credit-price and study-job log files from api server #5985

Closed bisgaard-itis closed 6 days ago

bisgaard-itis commented 1 week ago

What do these changes do?

image

N.B. @pcrespov and @wvangeit instead of downloading all logfiles from all nodes in a study job to the api-server, zip them and send them back to the user I decided to just forward the download links to the user. I will then add a function in the osparc client which does the download.

Related issue/s

How to test

Dev-ops checklist

codecov[bot] commented 6 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.7%. Comparing base (cafbf96) to head (bc764b9). Report is 295 commits behind head on master.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985/graphs/tree.svg?width=650&height=150&src=pr&token=h1rOE8q7ic&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation)](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) ```diff @@ Coverage Diff @@ ## master #5985 +/- ## ========================================= + Coverage 84.5% 87.7% +3.1% ========================================= Files 10 1415 +1405 Lines 214 58095 +57881 Branches 25 1340 +1315 ========================================= + Hits 181 50978 +50797 - Misses 23 6828 +6805 - Partials 10 289 +279 ``` | [Flag](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) | Coverage Δ | | |---|---|---| | [integrationtests](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) | `63.6% <ø> (?)` | | | [unittests](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) | `85.8% <100.0%> (+1.3%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) | Coverage Δ | | |---|---|---| | [...-server/src/simcore\_service\_api\_server/api/root.py](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?src=pr&el=tree&filepath=services%2Fapi-server%2Fsrc%2Fsimcore_service_api_server%2Fapi%2Froot.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#diff-c2VydmljZXMvYXBpLXNlcnZlci9zcmMvc2ltY29yZV9zZXJ2aWNlX2FwaV9zZXJ2ZXIvYXBpL3Jvb3QucHk=) | `100.0% <100.0%> (ø)` | | | [...c/simcore\_service\_api\_server/api/routes/credits.py](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?src=pr&el=tree&filepath=services%2Fapi-server%2Fsrc%2Fsimcore_service_api_server%2Fapi%2Froutes%2Fcredits.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#diff-c2VydmljZXMvYXBpLXNlcnZlci9zcmMvc2ltY29yZV9zZXJ2aWNlX2FwaV9zZXJ2ZXIvYXBpL3JvdXRlcy9jcmVkaXRzLnB5) | `100.0% <100.0%> (ø)` | | | [...vice\_api\_server/api/routes/solvers\_jobs\_getters.py](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?src=pr&el=tree&filepath=services%2Fapi-server%2Fsrc%2Fsimcore_service_api_server%2Fapi%2Froutes%2Fsolvers_jobs_getters.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#diff-c2VydmljZXMvYXBpLXNlcnZlci9zcmMvc2ltY29yZV9zZXJ2aWNlX2FwaV9zZXJ2ZXIvYXBpL3JvdXRlcy9zb2x2ZXJzX2pvYnNfZ2V0dGVycy5weQ==) | `93.4% <100.0%> (ø)` | | | [...core\_service\_api\_server/api/routes/studies\_jobs.py](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?src=pr&el=tree&filepath=services%2Fapi-server%2Fsrc%2Fsimcore_service_api_server%2Fapi%2Froutes%2Fstudies_jobs.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#diff-c2VydmljZXMvYXBpLXNlcnZlci9zcmMvc2ltY29yZV9zZXJ2aWNlX2FwaV9zZXJ2ZXIvYXBpL3JvdXRlcy9zdHVkaWVzX2pvYnMucHk=) | `94.0% <100.0%> (ø)` | | | [...mcore\_service\_api\_server/models/schemas/studies.py](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?src=pr&el=tree&filepath=services%2Fapi-server%2Fsrc%2Fsimcore_service_api_server%2Fmodels%2Fschemas%2Fstudies.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#diff-c2VydmljZXMvYXBpLXNlcnZlci9zcmMvc2ltY29yZV9zZXJ2aWNlX2FwaV9zZXJ2ZXIvbW9kZWxzL3NjaGVtYXMvc3R1ZGllcy5weQ==) | `100.0% <100.0%> (ø)` | | | [...simcore\_service\_api\_server/services/director\_v2.py](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?src=pr&el=tree&filepath=services%2Fapi-server%2Fsrc%2Fsimcore_service_api_server%2Fservices%2Fdirector_v2.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#diff-c2VydmljZXMvYXBpLXNlcnZlci9zcmMvc2ltY29yZV9zZXJ2aWNlX2FwaV9zZXJ2ZXIvc2VydmljZXMvZGlyZWN0b3JfdjIucHk=) | `76.9% <100.0%> (ø)` | | | [...c/simcore\_service\_api\_server/services/webserver.py](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985?src=pr&el=tree&filepath=services%2Fapi-server%2Fsrc%2Fsimcore_service_api_server%2Fservices%2Fwebserver.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#diff-c2VydmljZXMvYXBpLXNlcnZlci9zcmMvc2ltY29yZV9zZXJ2aWNlX2FwaV9zZXJ2ZXIvc2VydmljZXMvd2Vic2VydmVyLnB5) | `99.6% <100.0%> (ø)` | | ... and [1367 files with indirect coverage changes](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5985/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation)