helm icon indicating copy to clipboard operation
helm copied to clipboard

Feature request: add possibility to specify a values file inside a packaged chart

Open yann-soubeyrand opened this issue 7 years ago • 41 comments

Hi!

Oftentimes, one need to have values specific to the environment where the chart will be deployed. A possible solution is to have a common values.yaml file and several values-<environment>.yaml files. Deploying a chart from the source directory is easy using helm install --values values-<environment>.yaml.

However, when installing a packaged chart (from a chart repository), one cannot specify to use the values-<environment>.yaml file packaged inside the chart. This feature request is about adding the possibility to do so.

A better solution would be to add the notion of environment as a first class citizen in Helm and automatically use the values-<environment>.yaml file if present, but this would involve a lot more work!

yann-soubeyrand avatar Oct 16 '18 09:10 yann-soubeyrand

How about helm fetch --untar the chart before passing the values file?

hickeyma avatar Oct 16 '18 15:10 hickeyma

How about helm fetch --untar the chart before passing the values file?

This is a possible solution but not very user friendly ;-)

yann-soubeyrand avatar Oct 16 '18 18:10 yann-soubeyrand

Hey @hickeyma ,

We wrote a tool that wraps helm and has a deploy command that does all the logic for you. Check out the docs: https://github.com/maorfr/orca/tree/master/docs/commands#deploy-chart

Could this be of any help for you?

maorfr avatar Oct 24 '18 07:10 maorfr

@maorfr Thanks for this.

@yann-soubeyrand could this tool be of use?

hickeyma avatar Oct 25 '18 10:10 hickeyma

Yes, this tool may bring a solution to our use case, but having to use another tool "just" for this is not ideal in my opinion ;-) I think this should be a feature of Helm itself. Anyway, I'll have an eye on Orca to see if we can benefit from other features of the tool as well.

yann-soubeyrand avatar Oct 25 '18 13:10 yann-soubeyrand

Sure. I am not a core maintainer but @bacongobbler who is, has added the "feature" tag to this, so it might make it into a release at some stage.

hickeyma avatar Oct 25 '18 13:10 hickeyma

cc: @mattfarina

I propose that the following syntax to load values file from inside a chart archive.

--values chart:///values-production.yaml

This was all the existing behavior can be preserved. Do you think this is acceptable to add in Helm 3?

tamalsaha avatar Apr 29 '20 17:04 tamalsaha

the URL syntax looks wonky, but the premise sounds fine. I'd also suggest looking at other tickets asking for the same feature. Namely #6715

bacongobbler avatar May 14 '20 22:05 bacongobbler

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar Nov 22 '20 00:11 github-actions[bot]

@bacongobbler any update on this feature ? Referring to a specific value file inside a chart should be part of basic functionality IMO. If the value file derived from -f option is not found in the current path, attempt to find the value file from the downloaded chart tgz file. This issue is going to be closed soon as it is marked as stale (similar to https://github.com/helm/helm/issues/6715)

fritzb avatar Dec 20 '20 00:12 fritzb

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar Mar 22 '21 00:03 github-actions[bot]

bump!

bobbui avatar Mar 22 '21 04:03 bobbui

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar Jun 22 '21 00:06 github-actions[bot]

bump!

MaxRuenz avatar Jul 07 '21 08:07 MaxRuenz

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar Oct 06 '21 00:10 github-actions[bot]

bump!

bobbui avatar Oct 06 '21 18:10 bobbui

Hi, any updates about this feature? A very useful feature when need to test the chart with test valueFiles in chart.

awx-fuyuanchu avatar Oct 27 '21 01:10 awx-fuyuanchu

yes please!

soujiro32167 avatar Nov 08 '21 22:11 soujiro32167

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar Feb 07 '22 00:02 github-actions[bot]

bump

dhanvi avatar Feb 07 '22 21:02 dhanvi

Since others have been asking... No progress has been made on this ticket by members of the community.

The first step to propose a new feature would be to write a HIP.

We intend HIPs to be the primary mechanisms for proposing major new features, for collecting community input on an issue, and for documenting the design decisions that have gone into the project. The proposal author is responsible for building consensus within the community and documenting dissenting opinions.

See Proposing an Idea for more information on this process.

bacongobbler avatar Feb 08 '22 17:02 bacongobbler

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar May 10 '22 00:05 github-actions[bot]

bump

soujiro32167 avatar May 10 '22 11:05 soujiro32167

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar Aug 09 '22 00:08 github-actions[bot]

bump :)

dhanvi avatar Aug 11 '22 10:08 dhanvi

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar Nov 10 '22 00:11 github-actions[bot]

bump

dhanvi avatar Nov 10 '22 08:11 dhanvi

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar Feb 09 '23 01:02 github-actions[bot]

bump

ncaro-fdz avatar Feb 09 '23 09:02 ncaro-fdz

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

github-actions[bot] avatar May 12 '23 00:05 github-actions[bot]