20183: Don't do a periodic priority-update check at startup.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

diff --git a/lib/controller/localdb/container.go b/lib/controller/localdb/container.go
index b87f9033d5ef370cd7ac76e380ca4d9c301716b7..81f257181b0de10acea925d1fd04e500c42abc5f 100644 (file)
--- 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")
+               }
        }
 }