Merge pull request #5747 from chrischdi/pr-fix-panic-e2e

k8s-ci-robot · web-flow · commit c0ef9f4aa8ce · 2025-11-17T14:17:38.000-08:00
🌱 e2e: fix panic when dumping CloudTrail logs
diff --git a/test/e2e/shared/aws.go b/test/e2e/shared/aws.go
@@ -919,6 +919,7 @@ func DumpCloudTrailEvents(e2eCtx *E2EContext) {
 		page, err := paginator.NextPage(context.TODO())
 		if err != nil {
 			fmt.Fprintf(GinkgoWriter, "Couldn't get AWS CloudTrail events: err=%v\n", err)
+			break
 		}
 		events = append(events, page.Events...)
 	}

Original file line number	Diff line number	Diff line change
`@@ -919,6 +919,7 @@ func DumpCloudTrailEvents(e2eCtx *E2EContext) {`
`919`	`919`	`page, err := paginator.NextPage(context.TODO())`
`920`	`920`	`if err != nil {`
`921`	`921`	`fmt.Fprintf(GinkgoWriter, "Couldn't get AWS CloudTrail events: err=%v\n", err)`
	`922`	`+ break`
`922`	`923`	`}`
`923`	`924`	`events = append(events, page.Events...)`
`924`	`925`	`}`