Skip to content

chore: add a "make bump-go-version" to handle Go bumps across all files#17090

Merged
machine424 merged 2 commits intoprometheus:mainfrom
machine424:bump_go
Sep 1, 2025
Merged

chore: add a "make bump-go-version" to handle Go bumps across all files#17090
machine424 merged 2 commits intoprometheus:mainfrom
machine424:bump_go

Conversation

@machine424
Copy link
Member

Which issue(s) does the PR fix:

Fixes #16446

Supersedes #16488

Does this PR introduce a user-facing change?

NONE

This should help with partial updates like in #17080, we currently need to bump the version in too many places, the script can help keep track of that.

run using make bump-go-version

@machine424 machine424 changed the title Bump go chore: add a "make bump-go-version" to handle Go bumps across all files Aug 26, 2025
Co-authored-by: machine424 <ayoubmrini424@gmail.com>

Signed-off-by: machine424 <ayoubmrini424@gmail.com>
@machine424
Copy link
Member Author

cc @SuperQ

Copy link
Contributor

@Duciwuci Duciwuci left a comment

Choose a reason for hiding this comment

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

Unfortunately you get for the Job Test with previous Go Version:

-      image: quay.io/prometheus/golang-builder:1.23-base
+      image: quay.io/prometheus/golang-builder:1.25-base

For this Job, it is important to stay at N-1 Version instead of bumping this Job to the latest version.

Signed-off-by: machine424 <ayoubmrini424@gmail.com>
@machine424
Copy link
Member Author

nice catch, I think the order is messed up, pushed a new version

@Duciwuci
Copy link
Contributor

Thanks. LGTM for me. I'm not a Maintainer though, so let's wait for further Feedback. Thanks for taking this!

@SuperQ SuperQ self-requested a review August 28, 2025 15:08
@Duciwuci
Copy link
Contributor

Duciwuci commented Sep 1, 2025

@machine424 ready to merge?

@machine424 machine424 merged commit 2cbeef6 into prometheus:main Sep 1, 2025
28 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.

automate Go version updates

4 participants