Feature #14018

arvados-cwl-runner should support setting properties on container requests

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

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

100%

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

Description

We want to associate searchable metadata with container requests. Add a feature to arvados-cwl-runner so that in --submit mode the user can specify additional properties for the runner container request.

Suggest a command line option such as --runner-properties which takes a YAML file that contains the properties to be set on the container request.


Subtasks

Task #18086: Review 14018-acr-set-container-propertiesResolvedPeter Amstutz


Related issues

Related to Arvados Epics - Story #17848: Improve a-c-r usabilityIn Progress07/01/202112/31/2021

History

#1 Updated by Peter Amstutz about 3 years ago

  • Status changed from New to In Progress

#2 Updated by Peter Amstutz about 3 years ago

  • Tracker changed from Bug to Feature
  • Status changed from In Progress to New

#3 Updated by Peter Amstutz about 3 years ago

  • Description updated (diff)

#4 Updated by Peter Amstutz 4 months ago

  • Target version deleted (To Be Groomed)

#5 Updated by Peter Amstutz 3 months ago

#6 Updated by Peter Amstutz 3 months ago

  • Category set to CWL
  • Subject changed from [CWL] arvados-cwl-runner should support tagging container requests using properties to arvados-cwl-runner should support setting properties on container requests

#7 Updated by Peter Amstutz 3 months ago

  • Target version set to 2021-09-01 sprint

#8 Updated by Peter Amstutz 2 months ago

  • Target version changed from 2021-09-01 sprint to 2021-09-15 sprint

#9 Updated by Peter Amstutz about 2 months ago

  • Assigned To set to Peter Amstutz

#10 Updated by Peter Amstutz about 2 months ago

  • Status changed from New to In Progress

#11 Updated by Peter Amstutz about 1 month ago

14018-acr-set-container-properties @ 93be5ba7e96f1a62f53aee8c1e4979a99d1a9d94

Add arv:ProcessProperties hint, which sets properties on the container request.

Adds tests.

https://ci.arvados.org/view/Developer/job/developer-run-tests/2684/

#12 Updated by Lucas Di Pentima about 1 month ago

  • There're some debug print() calls left behind.
  • It seems there's a file that is pending to be committed: tests/wf/submit_wf_process_properties.cwl (the tests failed because of this).

#13 Updated by Peter Amstutz about 1 month ago

Lucas Di Pentima wrote:

  • There're some debug print() calls left behind.
  • It seems there's a file that is pending to be committed: tests/wf/submit_wf_process_properties.cwl (the tests failed because of this).

Good catch!

14018-acr-set-container-properties @ 4325505873cf6da23c81ef2bbf3003777463e0df

https://ci.arvados.org/view/Developer/job/developer-run-tests/2686/

#14 Updated by Lucas Di Pentima about 1 month ago

LGTM, thanks!

#15 Updated by Peter Amstutz about 1 month ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF