Skip to content

Conversation

@sylvestre
Copy link
Contributor

test_follow_when_files_are_pointing_to_same_relative_file_and_file_stays_same_size is flaky

test_follow_when_files_are_pointing_to_same_relative_file_and_file_stays_same_size is flaky
@sylvestre sylvestre requested a review from BenWiederhake April 14, 2025 08:02
@sylvestre sylvestre changed the title tail: Ignore a test on selinux tail: Ignore a few tests on selinux Apr 14, 2025
@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)

@BenWiederhake
Copy link
Collaborator

BenWiederhake commented Apr 14, 2025

The PR does exactly what it says. Hooray!

However, I'm not convinced that skipping the test on SELinux is a good idea. After all, it seems that "follow" is inherently nondeterministic. So maybe we should change the test to accomodate for some spurious events, instead of disabling this test (and probably more tests in the future)?

@sylvestre
Copy link
Contributor Author

@BenWiederhake agreed but it will require time to do it and until then, we can disable them + open a new issue

@sylvestre sylvestre merged commit a37c4a6 into main Apr 15, 2025
138 checks passed
@sylvestre sylvestre deleted the sylvestre-patch-5 branch April 15, 2025 13:21
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