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

OperatorHub instructions result in ClusterRoleBindings that are ineffective #4

Open
kingdonb opened this issue Aug 2, 2021 · 3 comments

Comments

@kingdonb
Copy link

kingdonb commented Aug 2, 2021

Following the OperatorHub instructions literally results in the flux operator going in the operators namespace

https://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:

but wanted to open a separate issue to track it here, since it is definitely not a flux CLI problem.

@chanwit
Copy link
Member

chanwit commented Aug 3, 2021

Thank you Kingdon!
Only an idea I have at the moment to solve this problem is to have another operator to install Flux for us.

wdyt?

kingdonb pushed a commit that referenced this issue Aug 4, 2021
@kingdonb
Copy link
Author

kingdonb commented Aug 4, 2021

[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..."

@kingdonb
Copy link
Author

kingdonb commented Aug 4, 2021

Maybe you already saw: operator-framework/kubectl-operator#50 (comment)

... in an ideal world an operator should be able to be installed in any namespace and have its RBAC correctly bound based on the installation namespace. That's currently a limitation of OLM, so not much we can do about it right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants