[ARVADOS] updated: 1.1.4-751-gfb95e36d2

[Git user]

Summary of changes:
 services/crunch-dispatch-slurm/squeue.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  fb95e36d2f0525fc7e024854b1bb3705027ca779 (commit)
      from  0fe48142de81e20967eaac3972e35d115aaaa497 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit fb95e36d2f0525fc7e024854b1bb3705027ca779
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Mon Aug 6 16:07:06 2018 -0400

    13399: Don't spam logs about low priority jobs that are running.
    
    Once a job has started, its priority relative to other jobs doesn't
    matter to us anyway, and we already skip low-priority jobs in the
    "renice" procedure, so there's no reason to warn about this.
    
    Also change "mysterious low priority" reporting threshold from 1Mi to
    20K to match the other "nice/hold race" code.
    
    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 ccbe44487..20305ab90 100644
--- a/services/crunch-dispatch-slurm/squeue.go
+++ b/services/crunch-dispatch-slurm/squeue.go
@@ -193,7 +193,7 @@ func (sqc *SqueueChecker) check() {
 			// resolved the same way.
 			log.Printf("releasing held job %q (priority=%d, state=%q, reason=%q)", uuid, p, state, reason)
 			sqc.Slurm.Release(uuid)
-		} else if p < 1<<20 && replacing.wantPriority > 0 {
+		} else if state != "RUNNING" && p <= 2*slurm15NiceLimit && replacing.wantPriority > 0 {
 			log.Printf("warning: job %q has low priority %d, nice %d, state %q, reason %q", uuid, p, n, state, reason)
 		}
 	}

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


hooks/post-receive
--