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

Allow updating chart dependencies when packaging a chart #2128

Closed
seh opened this issue Mar 16, 2017 · 1 comment
Closed

Allow updating chart dependencies when packaging a chart #2128

seh opened this issue Mar 16, 2017 · 1 comment
Labels
feature help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.

Comments

@seh
Copy link
Contributor

seh commented Mar 16, 2017

At present when one runs helm package against a chart with entries in its requirements.yaml file but an empty charts directory (or even one partially populated), Helm fails with the following complaint:

Warning: is in requirements.yaml but not in the charts/ directory!

It would be helpful to be able to tell Helm to acquire any missing dependencies before packaging, since packaging can't proceed without doing so anyway. Per discussion in the "helm" Slack channel, consider adding a flag to the helm package command that would first run the equivalent behavior of helm dependency update.

Such eager updating can't be the new default behavior without surprising long-time users who are already used to the manual dependency updating requirement.

@technosophos technosophos modified the milestones: 2.4.0, 2.4.0-Triage Mar 16, 2017
@technosophos
Copy link
Member

The discussion in the slack channel basically suggested that we provide a helm package -u flag that would cause helm dep up to be run prior to the packaging logic.

@technosophos technosophos added feature help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. labels Mar 17, 2017
sushilkm pushed a commit to sushilkm/helm that referenced this issue Jun 5, 2017
Added -u flag that would cause helm dep up before running helm package

Fixes helm#2128
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 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

2 participants