Skip to content

Vendor latest swarmkit, containerd v1.6.1#43357

Merged
thaJeztah merged 3 commits intomoby:masterfrom
corhere:vendor-swarmkit-and-containerd-v1.6.1
Mar 11, 2022
Merged

Vendor latest swarmkit, containerd v1.6.1#43357
thaJeztah merged 3 commits intomoby:masterfrom
corhere:vendor-swarmkit-and-containerd-v1.6.1

Conversation

@corhere
Copy link
Copy Markdown
Contributor

@corhere corhere commented Mar 10, 2022

- What I did
Upgraded swarmkit to the latest master, and containerd (client library) to v1.6.1. The process was surprisingly painless; could that be an ominous sign?

- How I did it
Other than manually removing replace statements from vendor.mod, I used go get -modfile vendor.mod and go mod tidy -modfile vendor.mod -compat 1.17 to handle all the edits.

- How to verify it
CI

- Description for the changelog
@dperny would what enhancements and bugfixes are included in the swarmkit upgrade

- A picture of a cute animal (not mandatory but encouraged)

@corhere corhere force-pushed the vendor-swarmkit-and-containerd-v1.6.1 branch 2 times, most recently from 32534a3 to 31ee173 Compare March 10, 2022 22:09
corhere added 3 commits March 10, 2022 17:48
Signed-off-by: Cory Snider <csnider@mirantis.com>
...in preparation for upgrading containerd.

Signed-off-by: Cory Snider <csnider@mirantis.com>
Signed-off-by: Cory Snider <csnider@mirantis.com>
@corhere corhere force-pushed the vendor-swarmkit-and-containerd-v1.6.1 branch from 6e43f36 to b36fb04 Compare March 10, 2022 22:48
@olljanat
Copy link
Copy Markdown
Contributor

Afaik you need also update these to make sure that CI works correctly

: "${CONTAINERD_VERSION:=v1.5.10}"

and
ARG CONTAINERD_VERSION=1.5.10

@thaJeztah
Copy link
Copy Markdown
Member

There's a separate PR to update the binary; #42942

The containerd library/vendor code is ok to be newer than the binary (due to containerd's v1.x compatibility guarantee at the grpc API level); this also allows upgrading containerd for existing versions of docker engine

Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants