No-op when the pods are not ready (#2)

christopherzli · web-flow · commit 16a968a6769e · 2024-05-10T16:02:46.000-07:00
No-op when the pods are not ready instead of proceeding
tested in integration test cluster
diff --git a/main.go b/main.go
@@ -580,7 +580,7 @@ func (c *controller) sync(ctx context.Context) {
 
 				if err := c.waitForPod(ctx, podName); err != nil {
 					level.Warn(c.logger).Log("msg", "failed polling until pod is ready", "pod", podName, "duration", time.Since(start), "err", err)
-					continue
+					return
 				}
 
 				level.Debug(c.logger).Log("msg", "waited until new pod was ready", "pod", podName, "duration", time.Since(start))

Original file line number	Diff line number	Diff line change
`@@ -580,7 +580,7 @@ func (c *controller) sync(ctx context.Context) {`
`580`	`580`
`581`	`581`	`if err := c.waitForPod(ctx, podName); err != nil {`
`582`	`582`	`level.Warn(c.logger).Log("msg", "failed polling until pod is ready", "pod", podName, "duration", time.Since(start), "err", err)`
`583`		`- continue`
	`583`	`+ return`
`584`	`584`	`}`
`585`	`585`
`586`	`586`	`level.Debug(c.logger).Log("msg", "waited until new pod was ready", "pod", podName, "duration", time.Since(start))`