common-workflow-language / cwl-v1.2

Released CWL v1.2.1 specification
https://www.commonwl.org/v1.2/
Apache License 2.0
33 stars 22 forks source link

make timelimt tests not require workReuse #85

Open mr-c opened 3 years ago

mr-c commented 3 years ago

@tetron would this be okay for Arvados?

https://github.com/common-workflow-language/cwl-v1.2/commit/a6cb03b3cb67dee2ea644e94b1a57deef4d03c00#diff-bc8aedc9592506dd171277abda2c1207a86260b82508a3a3289cde557e2d5af9R8

tetron commented 3 years ago

I think Arvados includes the time limit in deciding whether something is the "same" command or not, so this isn't such an issue. We could fix cwltool to do the same. But in general disabling reuse for these tests would be fine.

mr-c commented 3 years ago

@tetron Oh, I just wanted to move it to hints from requirements; I forgot to copy that over from the other thread