Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Pending verification] No status when lacking cache permissions #1298

Open
thetechnick opened this issue Sep 23, 2024 · 5 comments
Open

[Pending verification] No status when lacking cache permissions #1298

thetechnick opened this issue Sep 23, 2024 · 5 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@thetechnick
Copy link

If cluster-scoped list & watch permissions are missing for types contained in the bundle image, operator-controller will fail to populate .status.
Intenal error also blocks deletion of ClusterExtension CR, due to non-handling of finalizer teardown.

Log example:

W0918 08:58:27.304878       1 reflector.go:561] pkg/mod/k8s.io/[email protected]/tools/cache/reflector.go:243: failed to list apps/v1, Kind=Deployment: deployments.apps is forbidden: User "system:serviceaccount:nginx:nginx-install" cannot list resource "deployments" in API group "apps" at the cluster scope
E0918 08:58:27.304923       1 reflector.go:158] "Unhandled Error" err="pkg/mod/k8s.io/[email protected]/tools/cache/reflector.go:243: Failed to watch apps/v1, Kind=Deployment: failed to list apps/v1, Kind=Deployment: deployments.apps is forbidden: User \"system:serviceaccount:nginx:nginx-install\" cannot list resource \"deployments\" in API group \"apps\" at the cluster scope" logger="UnhandledError"
@thetechnick thetechnick added the kind/bug Categorizes issue or PR as related to a bug. label Sep 23, 2024
@thetechnick
Copy link
Author

Could be fixed with #1282

@everettraven
Copy link
Contributor

Would you mind sharing the "version" you ran? If it was a release on GH, which one? if from main, which commit?

@thetechnick
Copy link
Author

@everettraven Should have mentioned that directly in the issue description.
Tested with the latest GH release v0.15.0.

@everettraven
Copy link
Contributor

Ah, I think that was cut just before I did a pretty large refactor of our caching logic. Would you mind verifying that you are not seeing this issue if you pull the latest changes from the main branch?

@thetechnick
Copy link
Author

Will do o7

@thetechnick thetechnick changed the title No status when lacking cache permissions [Pending verification] No status when lacking cache permissions Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

2 participants