Story #10681

[CWL] arvados-cwl-runner should support --name flag

Added by Tom Morris over 4 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
12/07/2016
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5

Description

The current scheme of naming all jobs the same as the CWL file that they're running makes it difficult to distinguish jobs from each other if, for example, one has many runs of the same workflow with different inputs.


Subtasks

Task #10685: Review 10681-cwl-name-flagResolvedTom Clegg

Task #10688: Review 10681-cwl-wf-name-flagResolvedPeter Amstutz

Associated revisions

Revision 3e6ad647
Added by Peter Amstutz over 4 years ago

Merge branch '10681-cwl-name-flag' closes #10681

Revision b8e3dea8
Added by Peter Amstutz over 4 years ago

Merge branch '10681-cwl-wf-name-flag' closes #10681

Revision a092ab09 (diff)
Added by Peter Amstutz over 4 years ago

Fix crunch_script.py to pass name arg, refs #10681

History

#1 Updated by Peter Amstutz over 4 years ago

  • Status changed from New to In Progress

#2 Updated by Tom Clegg over 4 years ago

Just nits,

Would this

"name": kwargs["name"] if kwargs.get("name") else shortname(tool.tool["id"]),

be better as

"name": kwargs.get("name", shortname(tool.tool["id"])),

?

Worth adding a test assertion somewhere for the default/previous name behavior?

LGTM

#3 Updated by Peter Amstutz over 4 years ago

Tom Clegg wrote:

Just nits,

Would this

"name": kwargs["name"] if kwargs.get("name") else shortname(tool.tool["id"]),

be better as

"name": kwargs.get("name", shortname(tool.tool["id"])),

No, because kwargs.get("name") when kwargs contains {"name": None} will yield None, not the default value.

Worth adding a test assertion somewhere for the default/previous name behavior?

The existing tests that don't specify --name already assert the previous behavior.

LGTM

#4 Updated by Peter Amstutz over 4 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:3e6ad647c3da4cb77299249d1fa25767dc66b281.

#5 Updated by Peter Amstutz over 4 years ago

  • Status changed from Resolved to Feedback

#6 Updated by Radhika Chippada over 4 years ago

Branch 10681-cwl-wf-name-flag

  • I think it would be useful to also test updating a workflow / template with name?
  • It seems like test_create and test_create_name repeat a log of the code; wondering if code can be refactored and reused than repeated?

LGTM one way or the other. Thanks.

#7 Updated by Peter Amstutz over 4 years ago

  • Status changed from Feedback to Resolved
  • % Done changed from 50 to 100

Applied in changeset arvados|commit:b8e3dea8979a7dbbdb62b8ecd47f7094a7f0863c.

Also available in: Atom PDF