e2e: don't panic asserting inside wait.Poll

aojea · aojea · commit 353118c0475c · 2024-12-11T06:28:32.000Z
diff --git a/test/e2e/network/dns_common.go b/test/e2e/network/dns_common.go
@@ -490,7 +490,7 @@ func assertFilesContain(ctx context.Context, fileNames []string, fileDir string,
 
 			if err != nil {
 				if ctx.Err() != nil {
-					framework.Failf("Unable to read %s from pod %s/%s: %v", fileName, pod.Namespace, pod.Name, err)
+					return false, fmt.Errorf("Unable to read %s from pod %s/%s: %v", fileName, pod.Namespace, pod.Name, err)
 				} else {
 					framework.Logf("Unable to read %s from pod %s/%s: %v", fileName, pod.Namespace, pod.Name, err)
 				}

Original file line number	Diff line number	Diff line change
`@@ -490,7 +490,7 @@ func assertFilesContain(ctx context.Context, fileNames []string, fileDir string,`
`490`	`490`
`491`	`491`	`if err != nil {`
`492`	`492`	`if ctx.Err() != nil {`
`493`		`- framework.Failf("Unable to read %s from pod %s/%s: %v", fileName, pod.Namespace, pod.Name, err)`
	`493`	`+ return false, fmt.Errorf("Unable to read %s from pod %s/%s: %v", fileName, pod.Namespace, pod.Name, err)`
`494`	`494`	`} else {`
`495`	`495`	`framework.Logf("Unable to read %s from pod %s/%s: %v", fileName, pod.Namespace, pod.Name, err)`
`496`	`496`	`}`