16139: Fix secondaryFile errors when running --submit --no-wait
authorPeter Amstutz <peter.amstutz@curii.com>
Fri, 7 Feb 2020 20:59:10 +0000 (15:59 -0500)
committerPeter Amstutz <peter.amstutz@curii.com>
Fri, 7 Feb 2020 21:11:40 +0000 (16:11 -0500)
commit6272cb13e54254ad5ab3da898716daaab3ddae6a
tree9760fa7ea9c32df6d50c6f2b8c50f44dd5944566
parent8039b0a469df007a282f321bd8349d9e47461a79
16139: Fix secondaryFile errors when running --submit --no-wait

When submitting, we want to preserve the original CWL version of the
document.

However, when it creates a RunnerContainer (a cwltool.Process) it
examines the input interface and expects it to be in the 1.1 data
model.  But if we preserve/reload the original document, it is still
in the 1.0 data model.  This causes

The code was deliberately overriding the CWL version in metadata to
make this work.  This results in the problem reported on this ticket.

The fix is to maintain both the updated and preserved documents, and
use them appropriately where they are expected.

refs #16139

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>
sdk/cwl/arvados_cwl/executor.py
sdk/cwl/arvados_cwl/runner.py