[ARVADOS] created: 1.3.0-2812-ge5423707b

[Git user]

        at  e5423707b3e05110b67de9c286b1fcfdcc65113f (commit)


commit e5423707b3e05110b67de9c286b1fcfdcc65113f
Author: Tom Clegg <tom at tomclegg.ca>
Date:   Fri Jul 24 15:47:31 2020 -0400

    16631: Don't start new containers on nodes in hold/drain state.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at tomclegg.ca>

diff --git a/lib/dispatchcloud/worker/pool.go b/lib/dispatchcloud/worker/pool.go
index 7f1e4bc4b..79af5a0cb 100644
--- a/lib/dispatchcloud/worker/pool.go
+++ b/lib/dispatchcloud/worker/pool.go
@@ -494,7 +494,7 @@ func (wp *Pool) StartContainer(it arvados.InstanceType, ctr arvados.Container) b
 	defer wp.mtx.Unlock()
 	var wkr *worker
 	for _, w := range wp.workers {
-		if w.instType == it && w.state == StateIdle {
+		if w.instType == it && w.state == StateIdle && w.idleBehavior == IdleBehaviorRun {
 			if wkr == nil || w.busy.After(wkr.busy) {
 				wkr = w
 			}

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


hooks/post-receive
--