Skip to content

Fix version flags in builds#402

Merged
mjudeikis merged 2 commits into
kbind-dev:mainfrom
mjudeikis:mjudeikis/fixversion
Nov 27, 2025
Merged

Fix version flags in builds#402
mjudeikis merged 2 commits into
kbind-dev:mainfrom
mjudeikis:mjudeikis/fixversion

Conversation

@mjudeikis

Copy link
Copy Markdown
Contributor

Summary

Fix version string when building with goreleaser

What Type of PR Is This?

/kind bug

Related Issue(s)

Fixes #

Release Notes

NONE

@mjudeikis mjudeikis requested a review from a team as a code owner November 27, 2025 08:01
xrstf
xrstf previously approved these changes Nov 27, 2025

@xrstf xrstf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jo

@mjudeikis

Copy link
Copy Markdown
Contributor Author

Strange panic... very strange

go-version: v1.25.4
- name: Delete non-semver tags
run: 'git tag -d $(git tag -l | grep -v "^v")'
- name: Set LDFLAGS

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we also change the version in GIT_VERSION env to match version from release?
it is pinned to v0.0.0 atm eg. kube-bind-v0.0.0-a8dfe3c

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you want to use this for the image or smth else?

Currently this is the version:

v1.34.0+kube-bind-v0.5.1-58-g18cec9fd6a50c3

@mjudeikis mjudeikis enabled auto-merge (squash) November 27, 2025 08:37
@mjudeikis mjudeikis disabled auto-merge November 27, 2025 08:39

@xrstf xrstf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

under protest, because that change to the isolation strategy feels homeopathic at best

@mjudeikis mjudeikis enabled auto-merge (squash) November 27, 2025 08:52
@mjudeikis mjudeikis merged commit 573bae5 into kbind-dev:main Nov 27, 2025
25 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants