Merge pull request #11205 from vincepri/smooth-forbidden

k8s-ci-robot · web-flow · commit bd832612bfa3 · 2024-09-19T17:53:01.000+01:00
🌱 test/framework: GetCAPIResources should warn on rbac issues
diff --git a/test/framework/alltypes_helpers.go b/test/framework/alltypes_helpers.go
@@ -69,6 +69,10 @@ func GetCAPIResources(ctx context.Context, input GetCAPIResourcesInput) []*unstr
 			if apierrors.IsNotFound(err) {
 				continue
 			}
+			if apierrors.IsForbidden(err) {
+				fmt.Printf("Warning: failed to list %s resources due to a rbac issue: %v", typeList.GroupVersionKind(), err)
+				continue
+			}
 			Fail(fmt.Sprintf("failed to list %q resources: %v", typeList.GroupVersionKind(), err))
 		}
 		for i := range typeList.Items {

Original file line number	Diff line number	Diff line change
`@@ -69,6 +69,10 @@ func GetCAPIResources(ctx context.Context, input GetCAPIResourcesInput) []*unstr`
`69`	`69`	`if apierrors.IsNotFound(err) {`
`70`	`70`	`continue`
`71`	`71`	`}`
	`72`	`+ if apierrors.IsForbidden(err) {`
	`73`	`+ fmt.Printf("Warning: failed to list %s resources due to a rbac issue: %v", typeList.GroupVersionKind(), err)`
	`74`	`+ continue`
	`75`	`+ }`
`72`	`76`	`Fail(fmt.Sprintf("failed to list %q resources: %v", typeList.GroupVersionKind(), err))`
`73`	`77`	`}`
`74`	`78`	`for i := range typeList.Items {`