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

Helm Repo does not exist #1342

Closed
ryan-dyer-sp opened this issue Jan 4, 2021 · 9 comments
Closed

Helm Repo does not exist #1342

ryan-dyer-sp opened this issue Jan 4, 2021 · 9 comments
Labels
helm kind/bug Categorizes issue or PR as related to a bug.

Comments

@ryan-dyer-sp
Copy link

What happened:
The helm repo indicated by documentation does not exist.

helm repo add kube-state-metrics https://kubernetes.github.io/kube-state-metrics
Error: looks like "https://kubernetes.github.io/kube-state-metrics" is not a valid chart repository or cannot be reached: failed to fetch https://kubernetes.github.io/kube-state-metrics/index.yaml : 404 Not Found

What you expected to happen:
I found multiple existing issues and PRs around this problem but they all appear to be closed or merged. They all appear to point to it being a CD issue. Can someone please do the needful to get this fixed?

@ryan-dyer-sp ryan-dyer-sp added the kind/bug Categorizes issue or PR as related to a bug. label Jan 4, 2021
@austinorth
Copy link

For reference, here are the other issues that have since been closed erroneously are #1331 and #1337. This issue breaks some scripts I have written for setting up k8s development environments.

Looking at the commit history, it looks like @lilic or @scottrigby may be able to help with this?

@austinorth
Copy link

For now, I have switched over to the Bitnami chart here which works for my setup. May work as a temporary solution for you @ryan-dyer-sp

@chrissng
Copy link

chrissng commented Jan 6, 2021

Repo owners,

The Kubernetes user community at large depends on the great work done by the developer community. KSM is a critical aspect of any Kubernetes cluster and hence it is expected that the user community treats this repository as the source code as well as the official helm repository.

So to a certain degree there is urgency to set up the helm repository and make it available for public consumption. I am nudging the owners to get it done for the benefit of the community as a whole.

Thanks!

@lilic lilic added the helm label Jan 6, 2021
@lilic
Copy link
Member

lilic commented Jan 6, 2021

I am not a mantainer of the Helm chart so sadly cannot help here. @mrueg or @tariq1890 might be able to help better here.

@brancz
Copy link
Member

brancz commented Jan 6, 2021

Sorry, but neither @lilic or myself use helm at all. We accepted the helm chart files into this repo under the condition that someone that is not us maintains them. We are under no obligation to do so just because this promise was broken.

If @tariq1890 can't maintain it, and nobody steps up to do so, then we will be removing it. We'd love to work with Bitnami (or anyone else who would like to step up) if they would like to maintain the chart as part of the kube-state-metrics upstream project, but there is no incentive for us to do so, and frankly we don't owe anybody anything. If companies fail to invest in open source, and only take, then that's nobodies fault but their own.

@mrueg
Copy link
Member

mrueg commented Jan 6, 2021

I am happy to help out here and continue maintaining it. @scottrigby has been helping out with setting up the chart in this repo. Currently, there are some issues with the prow integration on test-infra for gh-pages (see scott's comment here: #1325 (comment)). While work is being done to get the chart published, you can still use https://charts.helm.sh/stable and https://charts.helm.sh/stable/packages/kube-state-metrics-2.9.3.tgz No major changes were made to the chart since then and no new proper version of kube-state-metrics has been released.

@brancz @lilic let me know if you prefer a disclaimer on https://github.com/kubernetes/kube-state-metrics/blob/master/charts/kube-state-metrics/README.md that this is still WIP.

@lilic
Copy link
Member

lilic commented Jan 6, 2021

A WIP is a good start, agreed! Would be great to have that and clarify the location of the current chart as well while you all resolve it, thanks!!

mrueg added a commit to mrueg/kube-state-metrics that referenced this issue Jan 6, 2021
mrueg added a commit to mrueg/kube-state-metrics that referenced this issue Jan 6, 2021
mrueg added a commit to mrueg/kube-state-metrics that referenced this issue Jan 6, 2021
mrueg added a commit to mrueg/kube-state-metrics that referenced this issue Jan 6, 2021
@austinorth
Copy link

Thank you all so much for the super quick responses! Really appreciate all of your hard work. 🤗 @mrueg if there's any way I can help too, let me know! Much love.

scottrigby added a commit to scottrigby/kube-state-metrics that referenced this issue Jan 10, 2021
The chart github release was already created, but due to a k8s infra ci issue
the index could not be pushed to gh-pages branch.

See:
- kubernetes/test-infra#20426
- kubernetes#1345
- kubernetes#1342
- kubernetes#1325 (comment)

Signed-off-by: Scott Rigby <[email protected]>
@scottrigby
Copy link
Contributor

scottrigby commented Jan 10, 2021

#1347 is ready to be merged. Once the gh actions from merging that complete properly (it was able to push the index.yaml file), then we should be good to go on other chart PRs and can close this issue.

After that, we can merge #1335 to give users friendly info about the helm repo.

mrueg added a commit to mrueg/prometheus-community-helm-charts that referenced this issue Apr 15, 2021
mrueg pushed a commit to mrueg/prometheus-community-helm-charts that referenced this issue Apr 15, 2021
mrueg added a commit to mrueg/prometheus-community-helm-charts that referenced this issue Apr 25, 2021
mrueg pushed a commit to mrueg/prometheus-community-helm-charts that referenced this issue Apr 25, 2021
langecode pushed a commit to neticdk/helm-charts that referenced this issue Aug 12, 2021
langecode pushed a commit to neticdk/helm-charts that referenced this issue Aug 12, 2021
QuentinBisson pushed a commit to giantswarm/prometheus-community-helm-charts-upstream that referenced this issue Oct 5, 2021
QuentinBisson pushed a commit to giantswarm/prometheus-community-helm-charts-upstream that referenced this issue Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helm kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

7 participants