-
Notifications
You must be signed in to change notification settings - Fork 2.4k
bump runc to v1.2.4 #3616
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
bump runc to v1.2.4 #3616
Conversation
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.
cmd/go.mod
needs updating too?
Missed that. Fixed now |
As I said in the description:
|
This is mostly because of go version bump in go.mod, which requires newer golangci-lint. |
@kolyshkin apologies! |
@kolyshkin #3615 has now been merged. |
Perhaps I'm missing something here, but it doesn't make any sense as run_tests do not use arguments. Signed-off-by: Kir Kolyshkin <[email protected]>
Updated; probably no longer a draft but I need to see a CI run to make sure. |
This bumps a few runc deps, and removes a few that we were able to decouple from. Signed-off-by: Kir Kolyshkin <[email protected]>
this would be immensely helpful, thanks! edit: @mythi would love to see this CI run as well. |
@kolyshkin once you move it off draft, i am happy to try to merge this |
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.
/approve
/lgtm
Let's
switch to go1.22 as the minimally supported version;(obsoleted by PR bump containerd related deps: api v1.8.0; errdefs v1.0.0; ttrpc v1.2.6 #3622)add go1.23 to CI, remove go1.21 (no longer supported);((obsoleted by PR bump containerd related deps: api v1.8.0; errdefs v1.0.0; ttrpc v1.2.6 #3622)Currently a draft pending #3615 merge. I'll rebase once that one is merged.Fixes: #3513