-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[release-17.0] Upgrade the Golang version to go1.21.10
#15863
Conversation
Signed-off-by: GitHub <noreply@github.com>
Hello! 👋 This Pull Request is now handled by arewefastyet. The current HEAD and future commits will be benchmarked. You can find the performance comparison on the arewefastyet website. |
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
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.
Conditionally approving this, the changes look good but the bootstrap image has not been pushed yet as the base Golang image is not fully available yet, thus a couple of CI workflows will fail until I push the new bootstrap images.
~/vitess# docker pull golang:1.21.10-bullseye
1.21.10-bullseye: Pulling from library/golang
no matching manifest for linux/amd64 in the manifest list entries
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
87fdd72
to
7afaf57
Compare
This Pull Request bumps the Golang version to
go1.21.10
and the bootstrap version to18.9
.There are a few manual steps remaining:
./.github/workflows/*.yml
files with the newer Golang version, the bot cannot handle that due to permissions.go run ./go/tools/go-upgrade/go-upgrade.go upgrade workflows --go-to=1.21.10
cc @vitessio/release