commit d4e5357d011f73190eebdc68effbb6222fe1a61a
author Han-Wen Nienhuys <hanwen@google.com> Mon Jan 27 20:34:27 2020 +0100
committer Han-Wen Nienhuys <hanwen@google.com> Wed Jan 29 20:38:09 2020 +0100
tree 08c5dbd6894023446328b9a2a152778f541baff8
parent 23e4268f54bd6727afd628720579e5d4612f00c2

Rate limit PendingChecksByScheme calls on failure

Change-Id: I75a0cdbeaa211cb3440776014bbc5f7908868351

cmd/checker/checker.go

diff --git a/cmd/checker/checker.go b/cmd/checker/checker.go
index 0fad572..10507ca 100644
--- a/cmd/checker/checker.go
+++ b/cmd/checker/checker.go
@@ -191,11 +191,15 @@
 // execute. It should be executed in a goroutine.
 func (c *gerritChecker) pendingLoop() {
 	for {
+		// TODO: real rate limiting.
+		time.Sleep(10 * time.Second)
+
 		pending, err := c.server.PendingChecksByScheme(checkerScheme)
 		if err != nil {
 			log.Printf("PendingChecksByScheme: %v", err)
 			continue
 		}
+
 		if len(pending) == 0 {
 			log.Printf("no pending checks")
 		}
@@ -207,8 +211,6 @@
 				log.Println("too busy; dropping pending check.")
 			}
 		}
-		// TODO: real rate limiting.
-		time.Sleep(10 * time.Second)
 	}
 }