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

Failed to fetch helm chart #3217

Closed
mike-pisman opened this issue Sep 28, 2023 · 5 comments
Closed

Failed to fetch helm chart #3217

mike-pisman opened this issue Sep 28, 2023 · 5 comments

Comments

@mike-pisman
Copy link

mike-pisman commented Sep 28, 2023

Hi,

Sorry if I am doing something wrong, but I cannot figure out how to install the helm chart:

$ helm install dendrite dendrite/dendrite -n matrix
Error: INSTALLATION FAILED: failed to fetch https://matrix-org.github.io/dendrite/dendrite-0.13.4.tgz : 404 Not Found

The helm repository shows:

$ helm search repo dendrite
NAME             	CHART VERSION	APP VERSION	DESCRIPTION      
dendrite/dendrite	0.13.4       	0.13.3     	Dendrite Matrix Homeserver

Versions:
helm version: v3.12.2
GoVersion: 1.20.7
Kubernetes version : v1.28.1

@mike-pisman
Copy link
Author

mike-pisman commented Sep 28, 2023

Well, until the solution is posted, the workaround is to download archive from the release page and install chart from the file:

$ helm install -f values.yaml dendrite ./dendrite-0.13.4.tgz

@S7evinK
Copy link
Contributor

S7evinK commented Sep 29, 2023

Doesn't look like you did something wrong.
Updating the chart-releaser-action seems to also pulled in a "bug". afaict, it will always use --packages-with-index, even though it shouldn't. This now resulted in dendrite-0.13.4.tgz being pushed to the gh-pages branch and the index.yaml not pointing to the package in the release.

helm/chart-releaser-action#163 should fix it for the future, I think.

@mike-pisman
Copy link
Author

helm/chart-releaser-action#163 should fix it for the future, I think.

Thank you, hopefully this will resolve the issue with the action. I will keep this one open for now, at least until they pull your changes.

@wrenix
Copy link
Contributor

wrenix commented Sep 29, 2023

#3220 should solve it in the current state.

@S7evinK
Copy link
Contributor

S7evinK commented Nov 10, 2023

Since helm/chart-releaser-action#172 was merged, closing this, going to update CI on the next release.

@S7evinK S7evinK closed this as completed Nov 10, 2023
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

No branches or pull requests

3 participants