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: do not use 'tail' for skip-file-rwx-check test #2307

Merged

Conversation

adrianreber
Copy link
Member

Newer versions of 'tail' rely on inotify and after a restore 'tail' is unhappy with the state of inotify and just stops.

This replaces 'tail' with a minimal test in C which just keeps a file open to still be able to test how CRIU reacts if the file mode has changed.

The skip-file-rwx-check test sometimes fails if kill is slower than tail ending itself.

@codecov-commenter
Copy link

codecov-commenter commented Nov 30, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b17a73b) 70.62% compared to head (a80cec5) 70.61%.

❗ Current head a80cec5 differs from pull request most recent head f5cd01a. Consider uploading reports for the commit f5cd01a to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##           criu-dev    #2307      +/-   ##
============================================
- Coverage     70.62%   70.61%   -0.01%     
============================================
  Files           134      134              
  Lines         33316    33316              
============================================
- Hits          23528    23526       -2     
- Misses         9788     9790       +2     

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

scripts/ci/vagrant.sh Outdated Show resolved Hide resolved
@adrianreber adrianreber force-pushed the 2023-11-30-skip-rwx-test branch from 78253be to dabb6a0 Compare November 30, 2023 17:01
@adrianreber adrianreber force-pushed the 2023-11-30-skip-rwx-test branch 2 times, most recently from 712a06a to 2f7daf4 Compare December 1, 2023 08:43
@adrianreber adrianreber force-pushed the 2023-11-30-skip-rwx-test branch from 2f7daf4 to f5cd01a Compare December 1, 2023 17:29
Signed-off-by: Adrian Reber <areber@redhat.com>
@adrianreber adrianreber force-pushed the 2023-11-30-skip-rwx-test branch from f5cd01a to 7e4ce49 Compare December 20, 2023 07:35
@adrianreber
Copy link
Member Author

Failing alpine test is tracked here: #2313

@adrianreber adrianreber merged commit e7aca13 into checkpoint-restore:criu-dev Dec 21, 2023
36 of 40 checks passed
@adrianreber adrianreber deleted the 2023-11-30-skip-rwx-test branch December 21, 2023 07:50
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.

5 participants