Skip to content

chore: Bump Go min version to 1.24#17080

Merged
machine424 merged 3 commits intoprometheus:mainfrom
Duciwuci:bump-go-ci
Sep 9, 2025
Merged

chore: Bump Go min version to 1.24#17080
machine424 merged 3 commits intoprometheus:mainfrom
Duciwuci:bump-go-ci

Conversation

@Duciwuci
Copy link
Contributor

This PR bumps the Go-Images using in the Prometheus-CI. Since Golang released Version 1.25 in August, we can proceed also with the CI.

Does this PR introduce a user-facing change?

NONE

@machine424
Copy link
Member

machine424 commented Aug 26, 2025

Thanks for this, unfortunately, there are other places where the version needs to be bumped.
let's get #17090 merged first, then you can update your PR with a make bump-go-version

@Duciwuci
Copy link
Contributor Author

Thanks for this, unfortunately, there are other places when the version needs to be bumped.

let's get #17090 merged first, then you can update your PR with a make bump-go-version

Good to know though, I will adjust my PR then :)

@Duciwuci Duciwuci force-pushed the bump-go-ci branch 2 times, most recently from ee71d70 to 69da756 Compare September 1, 2025 12:59
@Duciwuci
Copy link
Contributor Author

Duciwuci commented Sep 1, 2025

@machine424 I used ur Make-Target here 😄
Updated my PR.

@machine424 machine424 changed the title Chore(CI): Bump Go Version chore: Bump Go min version to 1.24 Sep 1, 2025
@machine424
Copy link
Member

/prombench main

@prombot
Copy link
Contributor

prombot commented Sep 1, 2025

⏱️ Welcome to Prometheus Benchmarking Tool. ⏱️

Compared versions: PR-17080 and main

After the successful deployment (check status here), the benchmarking results can be viewed at:

Available Commands:

  • To restart benchmark: /prombench restart main
  • To stop benchmark: /prombench cancel
  • To print help: /prombench help

@Duciwuci
Copy link
Contributor Author

Duciwuci commented Sep 1, 2025

/prombench restart main

@prombot
Copy link
Contributor

prombot commented Sep 1, 2025

@Duciwuci is not a org member nor a collaborator and cannot execute benchmarks.

@machine424
Copy link
Member

@Duciwuci is not a org member nor a collaborator and cannot execute benchmarks.

the root go.mod was already bumped and that's what matters for the benchmark. No need to re-run it.

Also, please adjust

# Update go.mod files
to include the two go.mod that you discovered.

I assume that (some go.mod not being updated) was caught by

, otherwise we'd need to adjust that script as well.

Also, we can now get rid of the tools.go in https://github.com/prometheus/prometheus/tree/2cbeef6d9582d7dee2bbe493a7cb42bda8f37553/internal/tools using https://tip.golang.org/doc/go1.24#tools and defining then at the root go.mod. I you want to work on that in a follow up PR. (I can assign #16442 to you)

@Duciwuci
Copy link
Contributor Author

Duciwuci commented Sep 2, 2025

@Duciwuci is not a org member nor a collaborator and cannot execute benchmarks.

the root go.mod was already bumped and that's what matters for the benchmark. No need to re-run it.

Also, please adjust

# Update go.mod files

to include the two go.mod that you discovered.
I assume that (some go.mod not being updated) was caught by

, otherwise we'd need to adjust that script as well.
Also, we can now get rid of the tools.go in https://github.com/prometheus/prometheus/tree/2cbeef6d9582d7dee2bbe493a7cb42bda8f37553/internal/tools using https://tip.golang.org/doc/go1.24#tools and defining then at the root go.mod. I you want to work on that in a follow up PR. (I can assign #16442 to you)

Thanks, I will adjust your script first and the second script will catch it correctly afterwards.

Let's get some process on the update first before I take further tasks :D

Signed-off-by: Duciwuci <duciwuci@gmail.com>
Signed-off-by: Duciwuci <duciwuci@gmail.com>
Signed-off-by: Duciwuci <duciwuci@gmail.com>
@machine424
Copy link
Member

/prombench cancel
Nothing noteworthy

@prombot
Copy link
Contributor

prombot commented Sep 5, 2025

Benchmark cancel is in progress.

Copy link
Member

@machine424 machine424 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@machine424 machine424 merged commit 6a5abd6 into prometheus:main Sep 9, 2025
28 checks passed
@bboreham bboreham mentioned this pull request Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants