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

Support lookup functions in the helm template calls #1558

Open
voor opened this issue May 1, 2024 · 1 comment
Open

Support lookup functions in the helm template calls #1558

voor opened this issue May 1, 2024 · 1 comment
Labels
discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution enhancement This issue is a feature request priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.

Comments

@voor
Copy link

voor commented May 1, 2024

Describe the problem/challenge you have
More charts are starting to implement the helm lookup function which will check existing secrets for certain values and populate them -- this provides a convenience and form of error checking in certain charts to make sure values are populated and correct. This creates problems for kapp which expects the helm template calls to not interact with existing cluster data.

Describe the solution you'd like
Allow the helm template call to use the service account that exists for the App CR to potentially get some data off the cluster. This would require adding --dry-run=server to the helm template command and providing a kubeconfig that would be accessible to the client.

Anything else you would like to add:
I don't like this, but helm chart authors are starting to implement this function in a way that breaks certain functionality if it's not used.


Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@voor voor added carvel-triage This issue has not yet been reviewed for validity enhancement This issue is a feature request labels May 1, 2024
@carvel-bot carvel-bot added this to Carvel May 1, 2024
@voor
Copy link
Author

voor commented May 1, 2024

Example: the Bitnami Redis chart won't let you use an existing secret anymore as a values, since if you want to use an existing secret it will lookup the value in that secret and then use it in the rendering instead of just letting you put in the reference to the secret. See here: Bitnami Redis chart template

@renuy renuy added discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. and removed carvel-triage This issue has not yet been reviewed for validity labels May 17, 2024
@renuy renuy moved this to Unprioritized in Carvel May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution enhancement This issue is a feature request priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Projects
Status: Unprioritized
Development

No branches or pull requests

2 participants