9262: Stop setting sbatch --priority in crunch-dispatch-slurm.
authorBrett Smith <brett@curoverse.com>
Tue, 19 Jul 2016 13:07:02 +0000 (09:07 -0400)
committerBrett Smith <brett@curoverse.com>
Tue, 19 Jul 2016 13:07:02 +0000 (09:07 -0400)
commitc0eb54eaffd1dac69f4ea73742a69a2473669538
tree9c04f17bdb585821d0b0cb4cb349a17ad5c870d0
parent779c5400a4f376489f4abc8564f890f309ae3f20
9262: Stop setting sbatch --priority in crunch-dispatch-slurm.

There are a few reasons we want to back this out:

* `--priority` is a relatively new feature of SLURM, added in 14.03,
  and it's not clear how many users are running that.
* Setting `--priority` requires extra privileges, per the sbatch man
  page, and it's not clear crunch-dispatch-slurm should run with
  those.
* There are multiple ways to set the "priority" of a SLURM job.
  There's also a niceness value.  Which is correct to use is probably
  ultimately going to depend on site rules or conventions.  We need to
  get more feedback on what people are doing before we implement
  something.  We might need configurations options along the lines of
  "translate container priority into this option with this scaling
  factor," or something like that.

Per discussion with Tom Clegg.  Closes #9626.
services/crunch-dispatch-slurm/crunch-dispatch-slurm.go
services/crunch-dispatch-slurm/crunch-dispatch-slurm_test.go