[ARVADOS] created: 1.1.4-160-g2c43723

[Git user]

        at  2c437234e2b72d0249d45f0ed3498bf4f1fa8f99 (commit)


commit 2c437234e2b72d0249d45f0ed3498bf4f1fa8f99
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Thu Apr 26 14:54:24 2018 -0400

    13399: Log when a slurm job's priority is unrealistically low.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/services/crunch-dispatch-slurm/squeue.go b/services/crunch-dispatch-slurm/squeue.go
index ee79c6f..9514da8 100644
--- a/services/crunch-dispatch-slurm/squeue.go
+++ b/services/crunch-dispatch-slurm/squeue.go
@@ -171,7 +171,10 @@ func (sqc *SqueueChecker) check() {
 			// reappeared, so rather than second-guessing
 			// whether SLURM is ready, we just keep trying
 			// this until it works.
+			log.Printf("releasing held job %q", uuid)
 			sqc.Slurm.Release(uuid)
+		} else if p < 1<<20 && replacing.wantPriority > 0 {
+			log.Printf("warning: job %q has low priority %d, nice %d, state %q, reason %q", uuid, p, n, state, reason)
 		}
 	}
 	sqc.queue = newq

-----------------------------------------------------------------------


hooks/post-receive
--