-
Notifications
You must be signed in to change notification settings - Fork 3.8k
[release/2.1] update release builds to 1.24.7 and add 1.25.1 to CI #12258
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
.github/workflows/ci.yml
Outdated
| matrix: | ||
| os: [ubuntu-22.04, ubuntu-24.04, ubuntu-24.04-arm, macos-13, windows-2022] | ||
| go-version: ["1.23.12", "1.24.6"] | ||
| go-version: ["1.24.6", "1.25.0"] |
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.
shouldn't CI continue to test the minimum version?
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.
We are changing the minimum version to 1.24.x since 1.23 is now EOL. Should we update the go.mod too?
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.
Why are we changing minimum versions in release branches?
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.
Since we dont want to keep an EOL go version as the minimal one.
Also, 2.1 has almost another year of support, if we decide not to change the minimum version in release branches, 1.23 will have to be maintained in CI for a long time.
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.
cc: @thaJeztah Any further thoughts on this?
4399d70 to
1e07c56
Compare
cpuguy83
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.
Looks like the commit (and pr title) are inaccurate since this is bumping to 1.24.
1e07c56 to
669b33f
Compare
- upgrade release builds from go1.23.12 to go1.24.7 - add 1.25.1 to CI NOTE: 1.23.12 is maintained in the CI, since go1.23 is the minimum version supported by containerd Signed-off-by: Akhil Mohan <akhilerm@gmail.com>
669b33f to
c54585b
Compare
Uh oh!
There was an error while loading. Please reload this page.