Bug #17333

arvados-cwl-runner does not validate that project-uuid is valid, and returns confusing error message if not

Added by Ward Vandewege 9 months ago. Updated 4 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
CWL
Target version:
Start date:
06/21/2021
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

If you give arvados-cwl-runner a --project-uuid argument that is not a valid project uuid (for example, you give it a collection uuid), then it errors out with a very confusing message:

INFO Name conflict on 'Downloads files from URL(s) input (3e987fd3b2f7601a8966389abf5ad892+123)', existing collection has an unexpected portable data hash
ERROR Unhandled error:
  <HttpError 403 when requesting https://2xpu4.arvadosapi.com/arvados/v1/collections?ensure_unique_name=true&alt=json returned "request failed: http://localhost:8004/arvados/v1/collections: 403 Forbidden: Owner uuid must be set to User or Group (req-hgbkh6u2owgawb9bfgw4)">

Subtasks

Task #17457: Review 17333-cwl-check-project-uuidResolvedPeter Amstutz

Associated revisions

Revision 29f67797
Added by Peter Amstutz 4 months ago

Merge branch '17333-cwl-check-project-uuid' refs #17333

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <>

History

#1 Updated by Ward Vandewege 9 months ago

  • Description updated (diff)

#2 Updated by Peter Amstutz 8 months ago

  • Target version set to 2021-03-31 sprint
  • Assigned To set to Jiayong Li

#3 Updated by Peter Amstutz 8 months ago

  • Category set to CWL
  • Subject changed from [arvados-cwl-runner] does not validate that project-uuid is valid, and returns confusing error message if not to arvados-cwl-runner does not validate that project-uuid is valid, and returns confusing error message if not

#4 Updated by Peter Amstutz 7 months ago

  • Target version changed from 2021-03-31 sprint to 2021-04-14 sprint

#5 Updated by Peter Amstutz 6 months ago

  • Target version changed from 2021-04-14 sprint to 2021-04-28 bughunt sprint

#6 Updated by Peter Amstutz 6 months ago

  • Target version deleted (2021-04-28 bughunt sprint)

#7 Updated by Peter Amstutz 4 months ago

  • Target version set to 2021-07-07 sprint

#8 Updated by Peter Amstutz 4 months ago

  • Target version changed from 2021-07-07 sprint to 2021-06-23 sprint

#9 Updated by Peter Amstutz 4 months ago

  • Assigned To changed from Jiayong Li to Peter Amstutz

#10 Updated by Peter Amstutz 4 months ago

  • Status changed from New to In Progress

#12 Updated by Lucas Di Pentima 4 months ago

This LGTM, thanks!

#13 Updated by Peter Amstutz 4 months ago

  • Status changed from In Progress to Resolved

#14 Updated by Peter Amstutz 4 months ago

  • Release set to 39

Also available in: Atom PDF