Project

General

Profile

Actions

Idea #9626

closed

[Crunch2] c-d-slurm should avoid new --priority option

Added by Brett Smith almost 8 years ago. Updated almost 8 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
Start date:
07/18/2016
Due date:
Story points:
0.5

Description

crunch-dispatch-slurm translates the container's priority into the sbatch --priority option. According to the news, this option was added in 14.03.2, which is probably too new for us to require (it's not even included Ubuntu trusty, supported until 2019).

It's also not clear to me that we want it at all. The man page says, "Only Slurm operators and administrators can set the priority of a job." It's not clear to me that crunch-dispatch-slurm should need to count on having those privileges. Even if it happens to have them, I'm not sure it should use what is apparently such a blunt instrument.


Related issues

Related to Arvados - Idea #9187: [Crunch2] Update dispatcher to use new API supportResolvedPeter Amstutz05/02/2016Actions
Actions #1

Updated by Brett Smith almost 8 years ago

  • Subject changed from [Crunch2] c-d-slurm uses new --priority option to [Crunch2] c-d-slurm should avoid new --priority option
Actions #2

Updated by Brett Smith almost 8 years ago

  • Target version set to Arvados Future Sprints
Actions #3

Updated by Brett Smith almost 8 years ago

  • Status changed from New to In Progress
  • Assigned To set to Brett Smith
  • Target version changed from Arvados Future Sprints to 2016-07-20 sprint
  • Story points set to 0.5
Actions #4

Updated by Brett Smith almost 8 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:c0eb54eaffd1dac69f4ea73742a69a2473669538.

Actions #5

Updated by Brett Smith almost 8 years ago

Discussed with Tom and agreed to remove --priority. See the commit message for rationale.

Actions

Also available in: Atom PDF