-
Notifications
You must be signed in to change notification settings - Fork 564
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
Proposal: Template values files #97
Comments
Also, just want to throw this out there: https://github.com/hairyhenderson/gomplate
richer library of functions. pretty much everything sprig can do, but more! =) |
@osterman Thx for the info! gomplate looks promising :) Just to sync up with you, it looks like there is a momentum to let helmfile be close to helm's spec/behavior. That being said, I'm eager to providing options to use an alternative template engine like gomplate as long as it doesn't lock helmfile to single template engine, and the implementation is not so hard. Actually, I prefer jsonnet as suggested in #92 here but it is attracted to no one 🤣 WDYT? Any input is welcomed! |
Overall I would tend to stick with sprig unless we supported both. |
@sstarcher I was considering to support this simply like: releases:
- name: myrelease
values:
- myvalues.yaml This configuration would just instruct helmfile to render a temporary yaml file from Alternatives to the above is: releases:
- name: myrelease
values:
- template: myvalues.yaml.tmpl releases:
- name: myrelease
valuesTemplates:
- myvalues.yaml.tmpl This looks more explicit which I personally prefer, but I guess the momentum in helmfile community is not make it too explicit like that, similar to helm's own template(no .tmpl extension, no distinction between template vs regular yaml file). In short, keeping it simple at glance seems like the way to go. WDYT? |
I would transparently support it s in your first example. Run the templates across all values files. |
@sstarcher Thx. Sounds good. I've posted an enhanced example in #119 (comment). |
Resolves roboll#97
Just submitted #216 for this |
Resolves roboll#97
To limit the need to duplicate values it would be helpful to be able to template the values files.
The text was updated successfully, but these errors were encountered: