[arvados] updated: 2.5.0-226-gcd1719593

git repository hosting git at public.arvados.org
Mon Mar 6 14:33:02 UTC 2023


Summary of changes:
 lib/controller/localdb/container.go | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

       via  cd171959303236ba9a85e6524e6c40310203325b (commit)
      from  ba41b025030022f6becf48f569d74c36f2a45ce4 (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 cd171959303236ba9a85e6524e6c40310203325b
Author: Tom Clegg <tom at curii.com>
Date:   Mon Mar 6 09:31:28 2023 -0500

    20183: Don't do a periodic priority-update check at startup.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/lib/controller/localdb/container.go b/lib/controller/localdb/container.go
index b87f9033d..81f257181 100644
--- a/lib/controller/localdb/container.go
+++ b/lib/controller/localdb/container.go
@@ -38,16 +38,16 @@ func (conn *Conn) runContainerPriorityUpdateThread(ctx context.Context) {
 	log := ctxlog.FromContext(ctx).WithField("worker", "runContainerPriorityUpdateThread")
 	ticker := time.NewTicker(5 * time.Minute)
 	for ctx.Err() == nil {
-		err := conn.containerPriorityUpdate(ctx, log)
-		if err != nil {
-			log.WithError(err).Warn("error updating container priorities")
-		}
 		select {
 		case <-ticker.C:
 		case <-conn.wantContainerPriorityUpdate:
 		case <-ctx.Done():
 			return
 		}
+		err := conn.containerPriorityUpdate(ctx, log)
+		if err != nil {
+			log.WithError(err).Warn("error updating container priorities")
+		}
 	}
 }
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list