-
Notifications
You must be signed in to change notification settings - Fork 76
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
feat(chart): added basic flux2-multi-tenancy chart #51
feat(chart): added basic flux2-multi-tenancy chart #51
Conversation
Signed-off-by: haarchri <[email protected]>
Signed-off-by: Daniel Werdermann <[email protected]>
Hmm this name is super confusing, what will happen to this chart when we'll implement: fluxcd/flux2#2093 ? |
Flux multi-tenancy can be implemented to with many other solutions e.g. OPA Gatekeep, at least please rename this to something that has Kyverno in the name. |
It was named so, because it is the implementation of https://github.com/fluxcd/flux2-multi-tenancy |
Should we rename it to |
Alternativly we can stick to it and use it in the future for other implementation as well. Loke OPA |
Are we going to delete this chart once RFC-0003 gets in? The flux2-multi-tenancy repo will use the new bootstrap option. |
Then we can adjust it (and have a major version bump) or delete it, If it gets totally obsolet. |
Cool 👍 |
Signed-off-by: haarchri [email protected]
What this PR does / why we need it:
Special notes for your reviewer:
Checklist