Merge pull request #5069 from AkihiroSuda/restart-fast

dmcgowan · web-flow · commit 9884730e5cad · 2021-02-25T16:37:53.000-08:00
restart: skip Sleep() for the first iteration of the reconcilation
diff --git a/runtime/restart/monitor/monitor.go b/runtime/restart/monitor/monitor.go
@@ -153,10 +153,10 @@ func (m *monitor) run(interval time.Duration) {
 		interval = 10 * time.Second
 	}
 	for {
-		time.Sleep(interval)
 		if err := m.reconcile(context.Background()); err != nil {
 			logrus.WithError(err).Error("reconcile")
 		}
+		time.Sleep(interval)
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -153,10 +153,10 @@ func (m *monitor) run(interval time.Duration) {`
`153`	`153`	`interval = 10 * time.Second`
`154`	`154`	`}`
`155`	`155`	`for {`
`156`		`- time.Sleep(interval)`
`157`	`156`	`if err := m.reconcile(context.Background()); err != nil {`
`158`	`157`	`logrus.WithError(err).Error("reconcile")`
`159`	`158`	`}`
	`159`	`+ time.Sleep(interval)`
`160`	`160`	`}`
`161`	`161`	`}`
`162`	`162`