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

helmfile deps is failing with Chart.lock: no such file or directory with helm 3.0.2 #1052

Closed
Nitive opened this issue Dec 24, 2019 · 3 comments · Fixed by #1056
Closed

helmfile deps is failing with Chart.lock: no such file or directory with helm 3.0.2 #1052

Nitive opened this issue Dec 24, 2019 · 3 comments · Fixed by #1056

Comments

@Nitive
Copy link

Nitive commented Dec 24, 2019

helmfile deps is failing with Chart.lock: no such file or directory with helm 3.0.2.
It happens because helmfile tries to access Chart.lock which is not there in 3.0.2.
The behavior was changes in helm/helm#7009 and now charts without apiVersion: v2 are treated as helm 2 carts and generate requirements.lock instead

Error:

in ./helmfile.yaml: unable to update deps: unable to resolve 1 deps: open /var/folders/k_/361cq75j2sgf6dxm6625422m0000gn/T/79305915 5/Chart.lock: no such file or directory

Steps to reproduce:

Repo with reproducible example

  1. Clone repo with example: git clone git@github.com:Nitive/helmfile-deps-helm-301-bug.git
  2. Install helm 3.0.1 and helm 3.0.2 with ./download-helm.sh linux-amd64 or ./download-helm.sh darwin-amd64
  3. Run helmfile deps with helm 3.0.1 — success
$ ./test-helm301.sh

Adding repo stable https://kubernetes-charts.storage.googleapis.com
"stable" has been added to your repositories

Updating repo
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "csssr" chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈ Happy Helming!⎈

Updating dependency /var/folders/k_/361cq75j2sgf6dxm6625422m0000gn/T/041766349
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "csssr" chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Downloading docker-registry from repo https://kubernetes-charts.storage.googleapis.com
Deleting outdated charts
  1. Run helmfile deps with helm 3.0.2 — error
$ ./test-helm302.sh

Adding repo stable https://kubernetes-charts.storage.googleapis.com
"stable" has been added to your repositories

Updating repo
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "csssr" chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈ Happy Helming!⎈

Updating dependency /var/folders/k_/361cq75j2sgf6dxm6625422m0000gn/T/793059155
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "csssr" chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Downloading docker-registry from repo https://kubernetes-charts.storage.googleapis.com
Deleting outdated charts

in ./helmfile.yaml: unable to update deps: unable to resolve 1 deps: open /var/folders/k_/361cq75j2sgf6dxm6625422m0000gn/T/793059155/Chart.lock: no such file or directory
@mumoshu
Copy link
Collaborator

mumoshu commented Dec 25, 2019

@Nitive Thanks a lot for the detailed report!

This should be fixed on the helmfile side.

mumoshu added a commit that referenced this issue Dec 27, 2019
mumoshu added a commit that referenced this issue Dec 27, 2019
@mumoshu
Copy link
Collaborator

mumoshu commented Dec 27, 2019

@Nitive I've fixed this in #1056 which is included since v0.98.1.

The example repository did help me a lot for reproducing the problem and testing the fix.

Thanks again for your support!

@Nitive
Copy link
Author

Nitive commented Dec 27, 2019

Thanks for a quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants