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: change sleep_test() and timestamp_test() to exclusive on Windows #4246

Merged
merged 2 commits into from
Jan 10, 2025

Conversation

bennylp
Copy link
Member

@bennylp bennylp commented Jan 10, 2025

pjlib-test's sleep_test() often fails on Windows GH CI, sleeping for longer than tolerated, even with --ci-mode. Similarly, timestamp_test() also often fails, drifting for more than tolerated. This probably depends on the load of GH runner, because sometimes it works reliably for many days.

As a workaround, this PR sets sleep_test() and timestamp_test() to run as exclusive test (=no other test will run while exclusive test is running) on Windows only.

Also while at it, set exclusive tests in pjlib-test to run last for slightly more efficient testing time

… run last for slightly more efficient testing time
@bennylp bennylp added this to the release-2.16 milestone Jan 10, 2025
@bennylp bennylp self-assigned this Jan 10, 2025
@bennylp bennylp changed the title CI: change sleep_test() as exclusive on Windows CI: change sleep_test() to exclusive on Windows Jan 10, 2025
@bennylp bennylp changed the title CI: change sleep_test() to exclusive on Windows CI: change sleep_test() and timestamp_test() to exclusive on Windows Jan 10, 2025
@bennylp bennylp merged commit 2d4b94a into master Jan 10, 2025
41 checks passed
@bennylp bennylp deleted the exclusive_sleep_test_ci branch January 10, 2025 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants