Project

General

Profile

Actions

Feature #14018

closed

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

Added by Peter Amstutz over 4 years ago. Updated about 1 year 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:
-
Release relationship:
Auto

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 1 (0 open1 closed)

Task #18086: Review 14018-acr-set-container-propertiesResolvedPeter Amstutz09/10/2021

Actions

Related issues

Related to Arvados Epics - Story #17848: CWL runner improvementsIn Progress07/01/202106/30/2023

Actions
Actions #1

Updated by Peter Amstutz over 4 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Peter Amstutz over 4 years ago

  • Tracker changed from Bug to Feature
  • Status changed from In Progress to New
Actions #3

Updated by Peter Amstutz over 4 years ago

  • Description updated (diff)
Actions #4

Updated by Peter Amstutz over 1 year ago

  • Target version deleted (To Be Groomed)
Actions #5

Updated by Peter Amstutz over 1 year ago

Actions #6

Updated by Peter Amstutz over 1 year 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
Actions #7

Updated by Peter Amstutz over 1 year ago

  • Target version set to 2021-09-01 sprint
Actions #8

Updated by Peter Amstutz over 1 year ago

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

Updated by Peter Amstutz over 1 year ago

  • Assigned To set to Peter Amstutz
Actions #10

Updated by Peter Amstutz over 1 year ago

  • Status changed from New to In Progress
Actions #11

Updated by Peter Amstutz over 1 year ago

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

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

Adds tests.

developer-run-tests: #2684

Actions #12

Updated by Lucas Di Pentima over 1 year 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).
Actions #13

Updated by Peter Amstutz over 1 year 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

developer-run-tests: #2686

Actions #14

Updated by Lucas Di Pentima over 1 year ago

LGTM, thanks!

Actions #15

Updated by Peter Amstutz over 1 year ago

  • Status changed from In Progress to Resolved
Actions #16

Updated by Peter Amstutz about 1 year ago

  • Release set to 42
Actions

Also available in: Atom PDF