-
Notifications
You must be signed in to change notification settings - Fork 2
build: bump go version to 1.25 #242
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
Conversation
Increments our minimum compatible go version to 1.25.0 and sets our toolchain version to 1.25.5. Also upgrades our tools to be compatible with the latest versions of `x/tools`. This resolves the errors from the `install-tools` target that several folks have run into.
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (18)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the 📝 WalkthroughWalkthroughGo toolchain and related developer tooling were bumped: CircleCI and Docker Go images updated to 1.25.5, go.mod/go directive/toolchain moved to Go 1.25 (go1.25.5), multiple module dependency upgrades, development tool pins updated, and NOTICE.txt license entries expanded. No control-flow or API changes. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In @.circleci/config.yml:
- Around line 40-41: The CircleCI config uses a non-existent Go version '1.25.5'
under the go/install step; update the version string in the .circleci config
(the go/install -> version entry) to the correct released Go version that
matches the project's go.mod toolchain directive (or change go.mod to the
intended released version), and ensure the value in go/install and the go.mod
`toolchain` directive remain synchronized.
In @docker/control-plane-dev/Dockerfile:
- Line 1: The Dockerfile uses a nonexistent base image tag "golang:1.25.5" in
the FROM instruction; replace that tag with a valid released Go image (for
example "golang:1.25" or "golang:1.24" or a specific patch tag listed on Docker
Hub) and verify the chosen tag exists on the official golang repository so
Docker builds will succeed.
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
.circleci/config.ymldocker/control-plane-dev/Dockerfilego.modtools.mk
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Codacy Static Code Analysis
🔇 Additional comments (2)
tools.mk (1)
14-21: All specified tool versions exist and are available on the Go module registry. The updates are valid:
gotestsumv1.13.0 ✓golangci-lint/v2v2.8.0 ✓ (released 2026-01-07)protobufv1.36.11 ✓goa/v3v3.23.4 ✓ (released 2025-12-14)goreleaser/v2v2.13.3 ✓syftv1.40.0 ✓ (released 2026-01-08)changiev1.24.0 ✓yamlfmtv0.21.0 ✓All versions are properly released and available. However, verify that golangci-lint v2.x configuration changes (if any) have been applied to the project's linter configuration before merging.
go.mod (1)
3-5: No action needed. Go 1.25.0 and go1.25.5 are valid Go releases (1.25 was released August 12, 2025, with 1.25.5 being the latest stable patch as of January 2026).Likely an incorrect or invalid review comment.
Upgrades goa in our go.mod to match the version in tools.mk.
8b90115 to
e98cafb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @go.mod:
- Around line 3-5: The go.mod currently sets the language version to the
vulnerable release "go 1.25.0" while the toolchain is pinned to "toolchain
go1.25.5"; update the go directive to a patched release (e.g., change the "go"
directive from 1.25.0 to 1.25.5 or at minimum 1.25.1) so the minimum required Go
matches the patched toolchain and avoids known CVEs.
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
go.sumis excluded by!**/*.sum
📒 Files selected for processing (2)
NOTICE.txtgo.mod
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Codacy Static Code Analysis
🔇 Additional comments (3)
NOTICE.txt (1)
2484-2485: NOTICE.txt dependency versions are correct and secure.All versions listed in NOTICE.txt match go.mod exactly. The critical dependencies—golang.org/x/crypto v0.46.0, google.golang.org/grpc v1.77.0, google.golang.org/protobuf v1.36.11, and golang.org/x/net v0.48.0—have no unpatched security vulnerabilities. License documentation is complete and valid.
go.mod (2)
47-47: The new indirect dependency is legitimate and verified.The
github.com/gohugoio/hashstructure v0.6.0dependency is valid and available on the Go module registry. No security advisories were found for this version.
147-158: All dependency versions are secure with no known vulnerabilities.Verified that
golang.org/x/crypto v0.46.0,golang.org/x/net v0.48.0, andgoogle.golang.org/grpc v1.77.0all exist and have no CVE advisories affecting them. These versions are well-patched and current.
rshoemaker
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
Increments our minimum compatible go version to 1.25.0 and sets our toolchain version to 1.25.5. Also upgrades our tools to be compatible with the latest versions of
x/tools. This resolves the errors from theinstall-toolstarget that several folks have run into.Testing
Summary by CodeRabbit
Chores
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.