-
Notifications
You must be signed in to change notification settings - Fork 124
Support lookup functions in the helm template calls #1558
Copy link
Copy link
Open
Labels
discussionThis issue is not a bug or feature and a conversation is needed to find an appropriate resolutionThis issue is not a bug or feature and a conversation is needed to find an appropriate resolutionenhancementThis issue is a feature requestThis issue is a feature requestpriority/awaiting-more-evidenceLowest priority. Possibly useful, but not yet enough support to actually get it done.Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Metadata
Metadata
Assignees
Labels
discussionThis issue is not a bug or feature and a conversation is needed to find an appropriate resolutionThis issue is not a bug or feature and a conversation is needed to find an appropriate resolutionenhancementThis issue is a feature requestThis issue is a feature requestpriority/awaiting-more-evidenceLowest priority. Possibly useful, but not yet enough support to actually get it done.Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Type
Projects
Status
Unprioritized
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 templatecalls to not interact with existing cluster data.Describe the solution you'd like
Allow the
helm templatecall 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=serverto 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.