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 chart should be added to hub.helm.sh #5962

Closed
kbreit opened this issue Jul 31, 2020 · 5 comments
Closed

Helm chart should be added to hub.helm.sh #5962

kbreit opened this issue Jul 31, 2020 · 5 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@kbreit
Copy link

kbreit commented Jul 31, 2020

I am was looking to upgrade from stable/nginx-ingress to kubernetes/ingress-nginx via Helm and didn't see the kubernetes/ingress-nginx at https://hub.helm.sh/. This repository should be added with possibly the stable/nginx-ingress repositories removed.

/kind feature

@kbreit kbreit added the kind/feature Categorizes issue or PR as related to a new feature. label Jul 31, 2020
@kfox1111
Copy link

I don't think they can remove stable/nginx-ingress from hub.helm.sh without removing all of stable.

+1 for having ingress-nginx listed in hub.helm.sh though.

@sharkymcdongles
Copy link
Contributor

@kfox1111 What is the difference between the stable chart and this one?

@davidkarlsen
Copy link
Member

@ChiefAlexander @kfox1111
I added some fixes to helm/charts#22823 so that it follows https://github.com/helm/charts/blob/master/PROCESSES.md - deprecated charts don't show up on helm hub. It would be great with a PR towards https://github.com/helm/hub to get this one listed there for visibility.

@scottrigby
Copy link
Contributor

@kfox1111 Yep you just need to index a chart marked with the deprecated flag and the hubs hide it. Thanks @davidkarlsen for the PR, the stable/nginx-ingress chart is now deprecated!

The upgrade path is now documented in the new ingress-nginx/ingress-nginx chart: #6038

@kbreit There's a PR in the Helm Hub for this: helm/hub#441

There's also now an initial entry at https://artifacthub.io/packages/search?page=1&repo=ingress-nginx (working to move that to a kubernetes organization there).

I think we can close this issue 😄

@aledbf aledbf closed this as completed Aug 26, 2020
@scottrigby
Copy link
Contributor

Update: moved to kubernetes org on Artifact Hub. Thanks @aledbf 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

6 participants