Skip to content
This repository has been archived by the owner on Feb 1, 2021. It is now read-only.

Update deps.sh #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update deps.sh #56

wants to merge 1 commit into from

Conversation

ercpereda
Copy link

Updating helm versions to avoid the issue:

Error: error initializing: Looks like "https://kubernetes-charts.storage.googleapis.com" is not a valid chart repository or cannot be reached: Failed to fetch https://kubernetes-charts.storage.googleapis.com/index.yaml : 403 Forbidden

https://helm.sh/blog/new-location-stable-incubator-charts/

Updating helm versions to avoid this issue
```
Error: error initializing: Looks like "https://kubernetes-charts.storage.googleapis.com" is not a valid chart repository or cannot be reached: Failed to fetch https://kubernetes-charts.storage.googleapis.com/index.yaml : 403 Forbidden
```

https://helm.sh/blog/new-location-stable-incubator-charts/
@james-gonzalez
Copy link

So upgrading this to the latest version of Helm will use the correct chart repo? We are having issues with this action now as well.

@ercpereda
Copy link
Author

So upgrading this to the latest version of Helm will use the correct chart repo? We are having issues with this action now as well.

Yes, that will fixed.

@james-gonzalez
Copy link

james-gonzalez commented Jan 12, 2021

@ercpereda - I think before its approved the CI check will need to pass.

Error: no repository definition for https://kubernetes-charts.storage.googleapis.com/. Please add the missing repos via 'helm repo add'

https://github.com/stefanprodan/hrval-action/pull/56/checks?check_run_id=1590861942#step:3:106

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

Successfully merging this pull request may close these issues.

2 participants