Feature #12526

Setting priority field through arvados-cwl-runner

Added by Bryan Cosca almost 2 years ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
10/31/2017
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5
Release relationship:
Auto

Description

I would like a --priority flag that will pass a priority number to the rest of the pipeline, for new jobs that spawn.


Subtasks

Task #13150: Add --priorityResolvedPeter Amstutz

Task #13151: Review 12526-cwl-priorityResolvedPeter Amstutz

Associated revisions

Revision f56ca98b
Added by Peter Amstutz over 1 year ago

Merge branch '12526-cwl-priority' closes #12526

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

Revision fdafe3f1 (diff)
Added by Peter Amstutz over 1 year ago

Fix arvados-cwl-runner --version argument refs #12526

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

Revision 4ad88f00 (diff)
Added by Peter Amstutz over 1 year ago

Fix arvados-cwl-runner --version argument (again) refs #12526

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

Revision 0ecd6b50 (diff)
Added by Peter Amstutz over 1 year ago

Fix "priority" KeyError in crunch1 refs #12526

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

History

#1 Updated by Tom Morris almost 2 years ago

  • Target version set to To Be Groomed

#2 Updated by Tom Morris almost 2 years ago

  • Story points set to 0.5

#3 Updated by Tom Morris almost 2 years ago

  • Target version changed from To Be Groomed to Arvados Future Sprints

#4 Updated by Tom Morris over 1 year ago

  • Assigned To set to Peter Amstutz
  • Target version changed from Arvados Future Sprints to 2018-03-14 Sprint

#5 Updated by Lucas Di Pentima over 1 year ago

  • File sdk/cwl/arvados_cwl/__init__.py - Lines 425 & 672: Do you think default priotity = 500 would be convenient to express it as a constant just in case it’s changed in the future?
  • Should the priority range be checked?

#6 Updated by Peter Amstutz over 1 year ago

Lucas Di Pentima wrote:

  • File sdk/cwl/arvados_cwl/__init__.py - Lines 425 & 672: Do you think default priotity = 500 would be convenient to express it as a constant just in case it’s changed in the future?

Fixed.

  • Should the priority range be checked?

Fixed.

12526-cwl-priority @ 6906707ffcbf0f063ab2a802b567f1f91a2fe84a

#8 Updated by Lucas Di Pentima over 1 year ago

This LGTM, please merge.

#9 Updated by Peter Amstutz over 1 year ago

  • Status changed from New to Resolved
  • % Done changed from 50 to 100

#10 Updated by Peter Amstutz over 1 year ago

  • Status changed from Resolved to Feedback

Eh, I broke the --version option with the fix for #12410. Oops.

#11 Updated by Peter Amstutz over 1 year ago

  • Status changed from Feedback to Resolved

Now it is fixed.

#12 Updated by Tom Morris about 1 year ago

  • Release set to 17

Also available in: Atom PDF