Merge pull request #260 from manavellamnimble/abortImagePullBackoff

divolgin · web-flow · commit 545313aa0665 · 2020-09-17T08:54:39.000-07:00
Abort run collector on ImagePullBackoff
diff --git a/pkg/collect/run.go b/pkg/collect/run.go
@@ -89,6 +89,13 @@ func runWithoutTimeout(ctx context.Context, c *Collector, pod *corev1.Pod, runCo
 			status.Status.Phase == corev1.PodSucceeded {
 			break
 		}
+		if status.Status.Phase == corev1.PodPending {
+			for _, v := range status.Status.ContainerStatuses {
+				if v.State.Waiting.Reason == "ImagePullBackOff" {
+					return nil, errors.Errorf("run pod aborted after getting pod status 'ImagePullBackOff'")
+				}
+			}
+		}
 		time.Sleep(time.Second * 1)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -89,6 +89,13 @@ func runWithoutTimeout(ctx context.Context, c Collector, pod corev1.Pod, runCo`
`89`	`89`	`status.Status.Phase == corev1.PodSucceeded {`
`90`	`90`	`break`
`91`	`91`	`}`
	`92`	`+ if status.Status.Phase == corev1.PodPending {`
	`93`	`+ for _, v := range status.Status.ContainerStatuses {`
	`94`	`+ if v.State.Waiting.Reason == "ImagePullBackOff" {`
	`95`	`+ return nil, errors.Errorf("run pod aborted after getting pod status 'ImagePullBackOff'")`
	`96`	`+ }`
	`97`	`+ }`
	`98`	`+ }`
`92`	`99`	`time.Sleep(time.Second * 1)`
`93`	`100`	`}`
`94`	`101`