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

ci: stop testing ubuntu overlayfs #2268

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

avagin
Copy link
Member

@avagin avagin commented Sep 24, 2023

They break it with each kernel rebase. More details are here: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1857257

Last time, it was fixed a few month ago and it has been broken again in 5.15.0-1046-azure.

@codecov-commenter
Copy link

Codecov Report

Patch coverage has no change and project coverage change: -0.04% ⚠️

Comparison is base (150eecc) 70.51% compared to head (67d2138) 70.48%.

❗ Current head 67d2138 differs from pull request most recent head 1607f0f. Consider uploading reports for the commit 1607f0f to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##           criu-dev    #2268      +/-   ##
============================================
- Coverage     70.51%   70.48%   -0.04%     
============================================
  Files           132      132              
  Lines         33507    33506       -1     
============================================
- Hits          23628    23616      -12     
- Misses         9879     9890      +11     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@adrianreber
Copy link
Member

So the fix is basically to bind mount the criu directory from the outside in the test container and being independent of the file system used by the container. Took me a while to spot that change. Would have been good to mention the change in the commit message.

Besides that both changed tests are happy. LGTM.

They break it with each kernel rebase. More details are here:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1857257

Last time, it was fixed a few month ago and it has been broken again in
5.15.0-1046-azure.

Let's bind-mount the CRIU directory into a test container to make it
independent of a container file system.

Signed-off-by: Andrei Vagin <avagin@gmail.com>
@avagin avagin merged commit 5e37ccf into checkpoint-restore:criu-dev Sep 25, 2023
26 of 36 checks passed
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.

3 participants