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

Publish the chart for alpha releases as an "dev" version #8047

Closed
jouve opened this issue Jul 13, 2023 · 3 comments
Closed

Publish the chart for alpha releases as an "dev" version #8047

jouve opened this issue Jul 13, 2023 · 3 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@jouve
Copy link

jouve commented Jul 13, 2023

What happened?

the helm chart containing an alpha appVersion is published as a stable version

What did you expect to happen?

the helm chart is published as a devel (as in needing helm --devel flag)

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

helm search repo kubernetes-dashboard

Anything else we need to know?

helm --devel is consistent with semver:

A pre-release version MAY be denoted by appending a hyphen and a series of dot separated identifiers immediately following the patch version. Identifiers MUST comprise only ASCII alphanumerics and hyphens [0-9A-Za-z-]. Identifiers MUST NOT be empty. Numeric identifiers MUST NOT include leading zeroes. Pre-release versions have a lower precedence than the associated normal version. A pre-release version indicates that the version is unstable and might not satisfy the intended compatibility requirements as denoted by its associated normal version. Examples: 1.0.0-alpha, 1.0.0-alpha.1, 1.0.0-0.3.7, 1.0.0-x.7.z.92, 1.0.0-x-y-z.--.

What browsers are you seeing the problem on?

No response

Kubernetes Dashboard version

na

Kubernetes version

na

Dev environment

No response

@jouve jouve added the kind/bug Categorizes issue or PR as related to a bug. label Jul 13, 2023
@floreks
Copy link
Member

floreks commented Jul 13, 2023

We can append chart version with alpha suffix too. I will have to check how hard it will be to update existing v7+ chart releases to also include this.

@floreks
Copy link
Member

floreks commented Aug 18, 2023

We have cleaned up all "stable" v7 releases and started over with the new v7.0.0-alpha0 release for the chart. I hope it won't cause too much trouble for the users.

/close

@k8s-ci-robot
Copy link
Contributor

@floreks: Closing this issue.

In response to this:

We have cleaned up all "stable" v7 releases and started over with the new v7.0.0-alpha0 release for the chart. I hope it won't cause too much trouble for the users.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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

3 participants