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

HIP: Helm Vault Integration for Secure Value Retrieval #387

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

Conversation

Vineet0197
Copy link

This HIP proposes integrating HashiCorp Vault with Helm to allow secure retrieval of sensitive values directly from Vault during Helm deployments. Currently, Helm users manually fetch secrets using external tools like the Vault CLI before passing them to Helm via --values. This approach introduces security risks and operational overhead.

By adding native Vault support in Helm, users can directly reference Vault secrets in their Helm commands, improving security, automation, and user experience.

Copy link
Contributor

@mattfarina mattfarina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the HIP. The idea generally makes sense. Because Helm cannot prioritize any one vendor over others, there needs to be some changes to the design. Helm cannot prioritize any one vendor because it is part of the CNCF. We need this to be accessible to all vendors. For the Helm teams maintaining, we would want this to be an extensible mechanism so we do not need to merge and maintain code for all vendors.

With that in mind, what about the location being an extension point like we have with downloader plugins? (see https://helm.sh/docs/topics/plugins/#downloader-plugins)

vault:// has a / in it which is invalid for paths on the operating systems that Helm supports. So, we can do a lookup for a plugin to handle this. Things like --vault-address can be part of the environment which is then passed to the plugin.

What do you think of that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants