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

Every version in the 7.x.y range got wiped from the Helm repository after merging #8103 #8191

Closed
WyriHaximus opened this issue Aug 18, 2023 · 2 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@WyriHaximus
Copy link

WyriHaximus commented Aug 18, 2023

What happened?

When iterating over a dashboard set up before lunch everything work, after lunch I'm now getting the following error:

Error: failed to fetch https://kubernetes.github.io/dashboard/kubernetes-dashboard-7.0.3.tgz : 404 Not Found

What did you expect to happen?

For helm to complete the installationHelm

How can we reproduce it (as minimally and precisely as possible)?

helm install my-kubernetes-dashboard k8s-dashboard/kubernetes-dashboard --version 7.0.3

Anything else we need to know?

Merging this change https://github.com/kubernetes/dashboard/pull/8103/files#diff-a5576e0c4e7c41a18e24d61963d76c69d9e181d8b9a88f87ce56842a922586a1R17 wiped out in the 7.x.y range and only leaves 7.0.0-alpha0 there as can be seen here: 5fba111 (Through #8190.) Note that this also wiped the tarballs so anyone with an old repo version can't use that anymore.

What browsers are you seeing the problem on?

No response

Kubernetes Dashboard version

n/a

Kubernetes version

1.27

Dev environment

kind

@WyriHaximus WyriHaximus added the kind/bug Categorizes issue or PR as related to a bug. label Aug 18, 2023
@floreks
Copy link
Member

floreks commented Aug 18, 2023

Unfortunately, this was intentional. We shouldn't release a stable chart version when the app was marked as an alpha in the first place and that was our mistake. It was brought to our attention by some of the users. In order to fix that, the only way was to clean up invalid "stable" chart releases and start over with the v7 alpha release. We are sorry for any issue it might have caused but to be in sync with the correct versioning scheme and app version, we've had to do this.

Ref: #8047

@WyriHaximus
Copy link
Author

Unfortunately, this was intentional. We shouldn't release a stable chart version when the app was marked as an alpha in the first place and that was our mistake. It was brought to our attention by some of the users. In order to fix that, the only way was to clean up invalid "stable" chart releases and start over with the v7 alpha release.

Thank you for clarifying why those versions were yanked. And yes that makes sense and I fully agree. It was a bit of an off moment, and I missed the updated helm chart versioning to start with alpha. When releasing the helm chart we will remove previous "official" releases without alpha suffixes. in #8103 now that I'm having a better look at it.

We are sorry for any issue it might have caused but to be in sync with the correct versioning scheme and app version, we've had to do this

Shit happens, going back to v6 for now 👍

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

No branches or pull requests

2 participants