🐛 Fixes mismatch on error status-code for start-computation

[pcrespov]

What do these changes do?

Mismatch between the error code specified in the OAS and the actual error code when starting a computation.

• the director-v2 service would respond with HTTP_403_FORBIDDEN for computation:start when the computation was started. Nonetheless, the OAS was reporting HTTP_409_CONFLICT.
• This error was propagated via the web-server and handled in the api-server.
• Due to the mismatch, the error was not handled properly in the api-server causing errors like this
• IMO HTTP_409_CONFLICT is more suitable than HTTP_403_FORBIDDEN since the latter is mostly related to authentication.
• Therefore, I changed the logic of the directorv2 to respond with HTTP_409_CONFLICT.
• Adapted also the front-end.
  • @odeimaiz I see you handle HTTP_422_UNPROCESSABLE_ENTITY as a non-error. Please double check the error codes in the OAS for that entrypoint. thx

Related issue/s

How to test

Dev-ops checklist

None

[odeimaiz]

@pcrespov yes, 422 is the status code we agreed for letting the users know that the pipeline is up-to-date, but they might still want to force a re-run.

Error code-wise might not be correct, but as long as all parts are in sync, we are fine.

[codecov[bot]]

Codecov Report

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

Project coverage is 85.1%. Comparing base (cafbf96) to head (2f2ae3f). Report is 298 commits behind head on master.

Additional details and impacted files

[](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5994?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) ```diff @@ Coverage Diff @@ ## master #5994 +/- ## ========================================= + Coverage 84.5% 85.1% +0.5% ========================================= Files 10 561 +551 Lines 214 28483 +28269 Branches 25 205 +180 ========================================= + Hits 181 24261 +24080 - Misses 23 4171 +4148 - Partials 10 51 +41 ``` | [Flag](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5994/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/5994/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) | `63.6% <100.0%> (?)` | | | [unittests](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5994/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) | `86.4% <100.0%> (+1.8%)` | :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/5994?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation) | Coverage Δ | | |---|---|---| | [...ore\_service\_director\_v2/api/routes/computations.py](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5994?src=pr&el=tree&filepath=services%2Fdirector-v2%2Fsrc%2Fsimcore_service_director_v2%2Fapi%2Froutes%2Fcomputations.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation#diff-c2VydmljZXMvZGlyZWN0b3ItdjIvc3JjL3NpbWNvcmVfc2VydmljZV9kaXJlY3Rvcl92Mi9hcGkvcm91dGVzL2NvbXB1dGF0aW9ucy5weQ==) | `83.7% <100.0%> (ø)` | | ... and [570 files with indirect coverage changes](https://app.codecov.io/gh/ITISFoundation/osparc-simcore/pull/5994/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ITISFoundation)