-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
build: update go version to 1.21 from 1.20 #12788
Conversation
209709d
to
8822fa8
Compare
go.mod
Outdated
@@ -1,6 +1,6 @@ | |||
module github.com/rook/rook | |||
|
|||
go 1.20 | |||
go 1.21 |
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.
I don't think we can update this until we drop 1.20 support.
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.
I think an earlier upgrade is better for us, or sometime soon some cve will be open to fix the golang version, do we need to wait for something before upgrading?
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.
It's good to support the most recent two versions of golang for a while so we can have some overlap to give devs and dependent CI systems time to update. Go 1.20 should still get CVE fixes for a while too.
8822fa8
to
5c5e2da
Compare
Signed-off-by: subhamkrai <srai@redhat.com>
5c5e2da
to
e9cc7ed
Compare
build: update go version to 1.21 from 1.20 (backport #12788)
Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:
skip-ci
on the PR.