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

feat(cluster) add ServiceAccountTemplate annotations and labels #423

Closed
wants to merge 4 commits into from

Conversation

Dariusch
Copy link
Contributor

@Dariusch Dariusch commented Oct 25, 2024

  • Updated the Helm plugin installation URL to https://github.com/KnechtionsCoding/helm-schema-gen.git and up-to-date fork of the original https://github.com/karuppiah7890/helm-schema-gen.git which is not compatible with macOS.
  • adding support for service account annotations and labels

fixes #309

@danpker
Copy link

danpker commented Nov 20, 2024

Would really like this feature, if there's anything I can do to help out, let me know!

@CopiiDeco
Copy link

Same here, would really need this

@itay-grudev itay-grudev added the chart( cluster ) Related to the cluster chart label Nov 29, 2024
@itay-grudev
Copy link
Collaborator

@Dariusch While your solution is perfectly valid and makes sense. I would like to pass the whole serviceAccountTemplate parameter in case it gets expanded in the future. I will mention you in my PR.

@itay-grudev
Copy link
Collaborator

Closed in favor of #463.

@itay-grudev itay-grudev closed this Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart( cluster ) Related to the cluster chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting up Annotation for service account in the Cluster chart
4 participants