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: helmfile build --embed-values to embed release values and secrets into the output #1436

Merged
merged 1 commit into from
Aug 29, 2020

Conversation

mumoshu
Copy link
Collaborator

@mumoshu mumoshu commented Aug 28, 2020

This is handy when debugging complex helmfile.yaml and values.yaml.gotmpl as the resulting helmfile.yaml contains all the rendered values and sops-encrypted secrets in its embedded form.

It's gonna be used to resolve a terraform-provider-helmfile issue that requires a unique hash of the desired state: mumoshu/terraform-provider-helmfile#28

Environment values and secrets are not yet supported. They are output as-is. But it should be fine as the resulting helm release won't change as long as release values/secrets are unchanged.

@mnatan
Copy link

mnatan commented Aug 28, 2020

Environment values and secrets are not yet supported. They are output as-is. But it should be fine as the resulting helm release won't change as long as release values/secrets are unchanged.

I don't think it's a problem for the terraform plugin, as it's only used for the diff cache SHA calculation and is not printed anywhere, right?

}

if len(paths) > 1 {
return nil, fmt.Errorf("glob patterns in release values and secrets is not supported yet. please submit a feature request if necessary")
Copy link

Choose a reason for hiding this comment

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

I like your error handling 😄

@mumoshu
Copy link
Collaborator Author

mumoshu commented Aug 29, 2020

I don't think it's a problem for the terraform plugin, as it's only used for the diff cache SHA calculation and is not printed anywhere, right?

@mnatan Thanks for reviewing ❤️ Yes, I think you're correct.

@mumoshu mumoshu merged commit 0fc0869 into master Aug 29, 2020
@mumoshu mumoshu deleted the build-with-values-embedding branch August 29, 2020 04:18
mumoshu added a commit that referenced this pull request Aug 29, 2020
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