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

Fixing build issue where wrong version is used #13059

Merged
merged 1 commit into from
May 23, 2024

Conversation

mattfarina
Copy link
Collaborator

The release process had selected the tag to use as the version automatically. But, this presented a problem when a release candidate and final release pointed to the same commit id. For a long time it had automatically selected the final release. But, we ran into a problem where it selected the RC tag instead of the final release.

This change explicitly tells the build scripts the version to use based on the tag passed into the CI run. It should no longer try to self discover the version.

Closes #13040

What this PR does / why we need it:

Special notes for your reviewer:

If applicable:

  • this PR contains documentation
  • this PR contains unit tests
  • this PR has been tested for backwards compatibility

The release process had selected the tag to use as the version
automatically. But, this presented a problem when a release
candidate and final release pointed to the same commit id. For a
long time it had automatically selected the final release. But,
we ran into a problem where it selected the RC tag instead of the
final release.

This change explicitly tells the build scripts the version to use
based on the tag passed into the CI run. It should no longer try
to self discover the version.

Closes helm#13040

Signed-off-by: Matt Farina <matt@mattfarina.com>
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 22, 2024
@mattfarina mattfarina added this to the 3.15.1 milestone May 23, 2024
@mattfarina mattfarina merged commit 691f313 into helm:main May 23, 2024
5 checks passed
@mattfarina mattfarina deleted the fix-build-tag branch May 23, 2024 14:05
@mattfarina mattfarina added the picked Indicates that a PR has been cherry-picked into the next release candidate. label May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
picked Indicates that a PR has been cherry-picked into the next release candidate. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Helm version for v3.15.0 reports v3.15.0-rc.2
3 participants