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

Toggle for ClusterRole and ClusterRoleBinding #39

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nessex
Copy link

@nessex nessex commented Dec 3, 2024

WHAT

Add toggles for the creation of ClusterRole and ClusterRoleBinding resources

WHY

Currently, these resources can't be conditionally disabled. In our case, we require these to be created in a different context. So adding these toggles will allow us to skip creating them as part of the helm install.

@macb
Copy link
Collaborator

macb commented Jan 7, 2025

Hey @nessex, sorry for the delay. Would you be willing to use a skip instead of create variable and flip the template logic? Would prefer for the current behavior to remain when the variable is not configured just to be safe.

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

Successfully merging this pull request may close these issues.

2 participants