Skip to content

ci: run cri-integration target for all runtimes#4686

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
samuelkarp:cri-tests-runtimes
Nov 3, 2020
Merged

ci: run cri-integration target for all runtimes#4686
dmcgowan merged 1 commit intocontainerd:masterfrom
samuelkarp:cri-tests-runtimes

Conversation

@samuelkarp
Copy link
Copy Markdown
Member

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Nov 2, 2020

Build succeeded.

@dims
Copy link
Copy Markdown
Member

dims commented Nov 3, 2020

@samuelkarp will this help with firecracker-containerd testing?

@samuelkarp
Copy link
Copy Markdown
Member Author

will this help with firecracker-containerd testing?

@dims Nope, this is just about making sure containerd's tests cover all three included runtimes (io.containerd.runtime.v1.linux, io.containerd.runc.v1, io.containerd.runc.v2).

(firecracker-containerd isn't compatible with CRI yet; there are still a number of known gaps.)

@dims
Copy link
Copy Markdown
Member

dims commented Nov 3, 2020

gotcha. thanks @samuelkarp

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit 8db2fe3 into containerd:master Nov 3, 2020
@samuelkarp samuelkarp deleted the cri-tests-runtimes branch November 3, 2020 19:14
@thaJeztah
Copy link
Copy Markdown
Member

1.4 cherry-pick for this is in containerd/cri#1603 (needs a re-vendor after that's merged)

@thaJeztah thaJeztah added cherry-picked/1.4.x PR commits are cherry picked into the release/1.4 branch and removed cherry-pick/1.4.x labels Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/1.4.x PR commits are cherry picked into the release/1.4 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants