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

Install issues with v3.0.0-beta.1 #6300

Merged
merged 1 commit into from Aug 28, 2019

Conversation

marckhouzam
Copy link
Member

What this PR does / why we need it:

I can no longer install charts with v3. Maybe I'm doing something wrong, or I'm running a different scenario?

The proposed fix makes sense to me as some new fields recently added had not been initialized.

Here's what I'm doing to trigger the problem:

# I start from an empty Helm Environment
> ls $HOME/Library/Caches/helm  $HOME/Library/Preferences/helm  $HOME/Library/helm
ls: /Users/marckhouzam/Library/Caches/helm: No such file or directory
ls: /Users/marckhouzam/Library/Preferences/helm: No such file or directory
ls: /Users/marckhouzam/Library/helm: No such file or directory

# Add a repo
> bin/helm repo add stable https://kubernetes-charts.storage.googleapis.com
"stable" has been added to your repositories

# Repo is there
> bin/helm repo list
NAME  	URL
stable	https://kubernetes-charts.storage.googleapis.com

# Correct directories have been modified
> ls $HOME/Library/Caches/helm  $HOME/Library/Preferences/helm  $HOME/Library/helm
ls: /Users/marckhouzam/Library/helm: No such file or directory
/Users/marckhouzam/Library/Caches/helm:
repository

/Users/marckhouzam/Library/Preferences/helm:
repositories.yaml

> bin/helm install nginx stable/nginx-ingress
Error: failed to download "stable/nginx-ingress" (hint: running `helm repo update` may help)

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

> bin/helm install nginx stable/nginx-ingress
Error: failed to download "stable/nginx-ingress" (hint: running `helm repo update` may help)

Signed-off-by: Marc Khouzam <marc.khouzam@ville.montreal.qc.ca>
@helm-bot helm-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 28, 2019
@FabianKramm
Copy link

Same as PR #6295, would be really great to see one of these PRs merged soon!

Copy link
Contributor

@hickeyma hickeyma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the fix @marckhouzam.

@hickeyma hickeyma added bug Categorizes issue or PR as related to a bug. v3.x Issues and Pull Requests related to the major version v3 labels Aug 28, 2019
@hickeyma hickeyma added this to the 3.0.0-beta.2 milestone Aug 28, 2019
@hickeyma hickeyma added this to In Progress in Helm 3 via automation Aug 28, 2019
@hickeyma hickeyma merged commit f46df92 into helm:dev-v3 Aug 28, 2019
Helm 3 automation moved this from In Progress to Done Aug 28, 2019
@hickeyma hickeyma changed the title Can no longer install charts with v3 Install issues with v3.0.0-beta.1 Aug 28, 2019
@hickeyma
Copy link
Contributor

Note: you can install charts from filesystem without this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Categorizes issue or PR as related to a bug. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. v3.x Issues and Pull Requests related to the major version v3
Projects
No open projects
Helm 3
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants