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

Specifying appVersion with "helm package" #2566

Closed
Bert-R opened this issue Jun 10, 2017 · 8 comments
Closed

Specifying appVersion with "helm package" #2566

Bert-R opened this issue Jun 10, 2017 · 8 comments
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.

Comments

@Bert-R
Copy link

Bert-R commented Jun 10, 2017

As I'd like to generate a helm package at build time, I would like to specify the appVersion as a command line parameter of the helm package command, just like you can do with the chart version.

Or is there an alternative way to do that? (I'm totally new to helm)

@seh
Copy link
Contributor

seh commented Jun 12, 2017

I thought I've seen a proposal for making the Chart.yaml file subject to some template expansion/interpolation, but I can't find any such issue today.

@technosophos technosophos added this to the Upcoming - Minor milestone Jun 12, 2017
@technosophos technosophos added the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Jun 12, 2017
@technosophos
Copy link
Member

Chart.yaml cannot be templated because it must be immutable from helm package time onward. But I think that adding a flag to set it at package time would be fine.

@Bert-R
Copy link
Author

Bert-R commented Jun 13, 2017

Indeed, that would match well with the --version flag. In addition, helm package does not have the --set option, which would be required when applying expansion/interpolation to Chart.yaml.

@adshmh
Copy link
Contributor

adshmh commented Nov 9, 2017

@bacongobbler I'd like to give this a try, as no one seems to be working on it.

@bacongobbler
Copy link
Member

@adshmh feel free to give this a shot! Nobody has begun work on this. :)

@sslavic
Copy link
Contributor

sslavic commented Nov 16, 2017

Requested in #3141 support for --values but supporting --set for helm package would be even more useful.

@adshmh
Copy link
Contributor

adshmh commented Nov 23, 2017

Just submitted a PR for app-version. Will work on --set next.

@Bert-R
Copy link
Author

Bert-R commented Nov 23, 2017

Thanks a lot Arash for working on this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.
Projects
None yet
Development

No branches or pull requests

7 participants