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

kustomize: generate namespaced and descoped rbacs #3165

Closed
lburgazzoli opened this issue Apr 3, 2022 · 9 comments
Closed

kustomize: generate namespaced and descoped rbacs #3165

lburgazzoli opened this issue Apr 3, 2022 · 9 comments
Assignees
Labels

Comments

@lburgazzoli
Copy link
Contributor

As today, a number of rbac can be found in the repo but when the operator should be installed in cluster mode, then all those rbca have to be translated from namespace to cluster (Role to ClusterRole, RoleBinding to ClòusterRoleBiding) which is of course possible but not very easy.

I wonder if we can generate them by default instead of having to use kustomize to transform them, i.e. we can add two additional folders:

  • config/rbac/namespaced
  • config/rbac/descoped
@lburgazzoli lburgazzoli added the area/installation Installation and Topology label Apr 3, 2022
@squakez
Copy link
Contributor

squakez commented Apr 5, 2022

cc @phantomjinx

@github-actions
Copy link
Contributor

github-actions bot commented Jul 5, 2022

This issue has been automatically marked as stale due to 90 days of inactivity.
It will be closed if no further activity occurs within 15 days.
If you think that’s incorrect or the issue should never stale, please simply write any comment.
Thanks for your contributions!

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale due to 90 days of inactivity.
It will be closed if no further activity occurs within 15 days.
If you think that’s incorrect or the issue should never stale, please simply write any comment.
Thanks for your contributions!

@gansheer
Copy link
Contributor

gansheer commented Nov 9, 2023

@lburgazzoli Do you mean having both version as files in the repo instead of generating "clustered" versions ?

@phantomjinx
Copy link
Contributor

@lburgazzoli Do you mean having both version as files in the repo instead of generating "clustered" versions ?

@gansheer I would say, yes, that is what @lburgazzoli means. There are advantages to it in that you don't then need to use the Makefile in order to leverage a kustomize cluster version install.

This is balanced by the usual disadvantage that more files are present in the repo and so have to be maintained. However, in this case I don't think this is a real problem.

@lburgazzoli
Copy link
Contributor Author

@gansheer yes correct, the main goal is to provide a simple kubectl apply -k https://github.com/apache/camel-k//config/global option to install camel-k

@gansheer
Copy link
Contributor

That fits the work I am doing refactoring kustomize.
@squakez I will work on this.

gansheer added a commit to gansheer/camel-k that referenced this issue Nov 10, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 13, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 13, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 13, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 13, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 14, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 15, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 16, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 17, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 20, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Nov 20, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Dec 5, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Dec 8, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Dec 11, 2023
gansheer added a commit to gansheer/camel-k that referenced this issue Dec 12, 2023
@squakez
Copy link
Contributor

squakez commented Apr 18, 2024

@gansheer I think this can be closed as completed, right?

@gansheer
Copy link
Contributor

Yes, I am closing it.

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

No branches or pull requests

4 participants