-
Notifications
You must be signed in to change notification settings - Fork 251
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
test_record, test_record_regex, test_record_all are flaky #732
Comments
Yeah - these tests have been a real problem. I'm hoping that some of the refactoring work @Karsten1987 is doing will make it a lot easier to construct these tests in a reliable way. We've got a few similar (but not identical) tickets in the backlog - I'd like to try and make a "code/test quality burndown board" to focus on when we feature-freeze rosbag2 on May 1. |
Also linux with cyclonedds_cpp is now failing more often than passing on repeated: https://ci.ros2.org/job/nightly_linux-aarch64_repeated/1593/testReport/(root)/projectroot/test_record__rmw_cyclonedds_cpp/history/ Also related #766 |
More instances here: Also potentially test_play |
Another example of The test timed out while running
Console output
|
#988 should improve the situation, I believe. |
|
Description
** clear and concise description of the bug is **
The three mentioned tests sometimes fail in CI jobs:
Expected Behavior
** clear and concise description of what you expected to happen **
Shouldn't fail.
Actual Behavior
** clear and concise description of what actually happened. include copied logs whenever possible **
The sometimes fail, particularly in the "repeated" jobs.
System (please complete the following information)
Additional context
** Add any other context about the problem here **
The text was updated successfully, but these errors were encountered: