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

Ability to add '--skip-refresh' to "helm dependency build" #368

Closed
cep21 opened this issue Nov 29, 2021 · 1 comment · Fixed by #396
Closed

Ability to add '--skip-refresh' to "helm dependency build" #368

cep21 opened this issue Nov 29, 2021 · 1 comment · Fixed by #396

Comments

@cep21
Copy link

cep21 commented Nov 29, 2021

FEATURE REQUEST

What happened:

When running chart-testing against multiple charts, the command helm dependency build X is run for each chart. On each run, I see the following text

Hang tight while we grab the latest from your chart repositories...

We have many helm chart repositories and this refresh on each chart we test in our mono repo is the slowest part of our CI/CD since it involves external network calls. Helm has an option, --skip-refresh, that can be passed to dependency build. This will skip this refresh step and greatly speed up CI/CD in helm mono repo setups.

What you expected to happen:

An option to tell chart-testing to run the helm refresh once, at the start of the command, then run with --skip-refresh on each sub chart. This will make CI/CD fast by not running refresh on each chart, but only at the start.

How to reproduce it (as minimally and precisely as possible):

Run chart-testing against multiple charts at once.

@nazarewk
Copy link
Contributor

nazarewk added a commit to nazarewk/chart-testing that referenced this issue Feb 18, 2022
nazarewk added a commit to nazarewk/chart-testing that referenced this issue Feb 18, 2022
nazarewk added a commit to nazarewk/chart-testing that referenced this issue Feb 18, 2022
nazarewk added a commit to nazarewk/chart-testing that referenced this issue Feb 18, 2022
nazarewk added a commit to nazarewk/chart-testing that referenced this issue Feb 18, 2022
addresses helm#368

Signed-off-by: Krzysztof Nazarewski <[email protected]>
nazarewk added a commit to nazarewk/chart-testing that referenced this issue Mar 14, 2022
addresses helm#368

Signed-off-by: Krzysztof Nazarewski <[email protected]>
nazarewk added a commit to nazarewk/chart-testing that referenced this issue Mar 14, 2022
addresses helm#368

Signed-off-by: Krzysztof Nazarewski <[email protected]>
nazarewk added a commit to nazarewk/chart-testing that referenced this issue Mar 23, 2022
addresses helm#368

Signed-off-by: Krzysztof Nazarewski <[email protected]>
cpanato pushed a commit that referenced this issue Apr 5, 2022
addresses #368

Signed-off-by: Krzysztof Nazarewski <[email protected]>
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 a pull request may close this issue.

2 participants