[ARVADOS] updated: 1.3.0-3041-gda85d6516
Git user
git at public.arvados.org
Mon Aug 31 19:52:32 UTC 2020
Summary of changes:
lib/dispatchcloud/worker/pool.go | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
via da85d6516630d06ae3c34b4a52dc5ddff9fd5ace (commit)
from baa1f256924655d67b704f35981e9839743fab99 (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 da85d6516630d06ae3c34b4a52dc5ddff9fd5ace
Author: Tom Clegg <tom at tomclegg.ca>
Date: Mon Aug 31 15:34:22 2020 -0400
16739: Fix using two different throttles.
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 2d958d837..ec6a049e6 100644
--- a/lib/dispatchcloud/worker/pool.go
+++ b/lib/dispatchcloud/worker/pool.go
@@ -170,9 +170,6 @@ type Pool struct {
runnerMD5 [md5.Size]byte
runnerCmd string
- throttleCreate throttle
- throttleInstances throttle
-
mContainersRunning prometheus.Gauge
mInstances *prometheus.GaugeVec
mInstancesPrice *prometheus.GaugeVec
@@ -307,7 +304,7 @@ func (wp *Pool) Create(it arvados.InstanceType) bool {
}
wp.mtx.Lock()
defer wp.mtx.Unlock()
- if time.Now().Before(wp.atQuotaUntil) || wp.throttleCreate.Error() != nil {
+ if time.Now().Before(wp.atQuotaUntil) || wp.instanceSet.throttleCreate.Error() != nil {
return false
}
// The maxConcurrentNodeCreateOps knob throttles the number of node create
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list