Bug #12186

[cwl] cwl.input.yml contains "nameroot" and "nameext" fields, breaks reuse with RunInSingleContainer

Added by Peter Amstutz 3 months ago. Updated about 1 month ago.

Status:ResolvedStart date:09/12/2017
Priority:NormalDue date:
Assignee:Peter Amstutz% Done:

100%

Category:-
Target version:2017-09-13 Sprint
Story points-Remaining (hours)0.00 hour
Velocity based estimate-

Subtasks

Task #12188: Review 12186-cwl-no-namerootResolvedPeter Amstutz

Associated revisions

Revision 194df852
Added by Peter Amstutz 2 months ago

Merge branch '12186-cwl-no-nameroot' closes #12186

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

History

#1 Updated by Peter Amstutz 3 months ago

  • Status changed from New to In Progress
  • Assignee set to Peter Amstutz

#2 Updated by Peter Amstutz 3 months ago

  • Target version changed from 2017-08-30 Sprint to 2017-09-13 Sprint

#4 Updated by Lucas Di Pentima 2 months ago

As far as I can tell, this looks good to me. One question: would it be too complex to make a test to prove that reuse is working with RunInSingleContainer? This would detect potential similar problems in the future.

#5 Updated by Peter Amstutz 2 months ago

Lucas Di Pentima wrote:

As far as I can tell, this looks good to me. One question: would it be too complex to make a test to prove that reuse is working with RunInSingleContainer? This would detect potential similar problems in the future.

I did this is 45054326217ef62eeef9bcdbd0ed429c55650609 I added a File input called "fileblub", and updated TestWorkflow.test_run to check that the intermediate input.cwl.json has only the expected fields for File objects.

#6 Updated by Peter Amstutz 2 months ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:194df852b504214879c005a0b730ab1e00395160.

#7 Updated by Tom Morris about 1 month ago

From a customer point of view, it's important for them to be able to know which jobs are potentially affected by this issue.

Do we know the range of versions and/or dates that are affected?

Also available in: Atom PDF