-
Notifications
You must be signed in to change notification settings - Fork 4
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
OperatorHub instructions result in ClusterRoleBindings that are ineffective #4
Comments
Thank you Kingdon! wdyt? |
Fixes #4 Signed-off-by: Kingdon Barrett <[email protected]>
[snip] This discussion was meant for the logs issue, I'm mixed up sorry. (Moving these notes over to the Draft PR.) I think we have enough operators already, we're going to need a "Yo dawg" meme in the documentation if there are any more. "I heard you like operators, so I put an operator in your operator lifecycle manager, then used it to install another operator..." |
Maybe you already saw: operator-framework/kubectl-operator#50 (comment)
|
Following the OperatorHub instructions literally results in the flux operator going in the
operators
namespacehttps://operatorhub.io/operator/flux
The ClusterRoleBindings have hardcoded references to
flux-system
for the service accounts that should be used as the default SA for Flux reconcilers. The result is some ineffective ClusterRoleBindings that do not grant any permission to any extant service accounts.I mentioned this in:
flux logs
not compatible with OLMflux
operator fluxcd/flux2#1673but wanted to open a separate issue to track it here, since it is definitely not a
flux
CLI problem.The text was updated successfully, but these errors were encountered: