Skip to content
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

Update containerd dependency to v1.4.9 #6485

Closed
wants to merge 2 commits into from

Conversation

dqminh
Copy link
Contributor

@dqminh dqminh commented Aug 20, 2021

This only upgrades the tracked dependencies of containerd to 1.4 from 1.3, tracking in #6232

This doesn't include any code changes in libraries usages or runtimeoptions to try to maintain compat with 1.3 as much as possible. As far as i can see, all containerd tests are passing locally.

dqminh added 2 commits August 20, 2021 17:32
This updates containerd dependencies to v1.4 with

```
go get github.com/containerd/[email protected]
bazel run //:gazelle -- update-repos -from_file=go.mod
```
Commit 453cb28c0b153e756b425ab34f5c2b0e6d57b6f2 update
honnef.co/go/tools in WORKSPACE but didn't update it in go.mod, so later
attempt to sync using gazelle will revert to older version which doesn't
build.

This is the result of

```
go get honnef.co/go/[email protected]
bazel run //:gazelle -- update-repos -from_file=go.mod
```
@google-cla google-cla bot added the cla: yes CLA has been signed label Aug 20, 2021
@avagin avagin requested a review from fvoznika August 21, 2021 08:55
Copy link
Member

@fvoznika fvoznika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Daniel, we need to wait for Kubernetes 1.17, which uses containerd 1.3, to reach end of life before we can start using 1.4. This link has the schedule for release deprecations. 1.17 end of life is set to 2022-01.

@dqminh
Copy link
Contributor Author

dqminh commented Aug 25, 2021

@fvoznika i think binaries built with containerd 1.4 should still work with 1.3 runtime as containerd protocol has not changed since v1.2, So this change should still work with containerd 1.3.

@fvoznika
Copy link
Member

@dqminh containerd is backwards compatible with older shims, e.g. containerd 1.3 works with shims built with 1.2. However, shims are not guaranteed to be backward compatible with older containerd, e.g. shim built with 1.3 interfacing with containerd 1.2.

@dqminh dqminh closed this Nov 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes CLA has been signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants