Skip to content

Re-enable criu in main integration runs#5614

Merged
mxpv merged 1 commit intocontainerd:mainfrom
estesp:test-criu-overlay
Jun 19, 2021
Merged

Re-enable criu in main integration runs#5614
mxpv merged 1 commit intocontainerd:mainfrom
estesp:test-criu-overlay

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Jun 16, 2021

Ubuntu kernel updates claim that the criu+overlayfs bug is finally fixed
in released kernels.

Signed-off-by: Phil Estes [email protected]

@k8s-ci-robot
Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Jun 16, 2021

Build succeeded.

@estesp
Copy link
Copy Markdown
Member Author

estesp commented Jun 16, 2021

Looks like it worked...but now need to special case the crun part of the matrix :-} (crun doesn't support the criu options like runc does apparently)

Ubuntu kernel updates claim that the criu+overlayfs bug is finally fixed
in released kernels.

Signed-off-by: Phil Estes <[email protected]>
@estesp estesp force-pushed the test-criu-overlay branch from 9df0706 to 394f864 Compare June 16, 2021 21:46
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Jun 16, 2021

Build succeeded.

@estesp estesp marked this pull request as ready for review June 17, 2021 15:30
@estesp estesp changed the title DRAFT: Re-enable criu in main integration runs Re-enable criu in main integration runs Jun 17, 2021
@estesp
Copy link
Copy Markdown
Member Author

estesp commented Jun 17, 2021

This is ready for review

@mxpv mxpv merged commit c88a5d0 into containerd:main Jun 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants