-
Notifications
You must be signed in to change notification settings - Fork 2
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
Flaky Windows test regressions: Access Violation
#9
Comments
Happened today in First
|
Windows Debug | Build regression | Investigating 🕵️Today's fail: https://ci.ros2.org/view/nightly/job/nightly_win_deb/2960/
Outputs for the last 15 failures:
Changes between first consistent fail:
Conclusions from this output:
Test using last FastDDS stable version: In the reproduced failure we can see:
The most probable option is that eProsima/Fast-DDS#4151 make the |
Thanks for doing some additional work here! That said, I don't think I agree with your conclusion. It looks to me like eProsima/Fast-DDS#4151 works just fine, and it is instead that eProsima/Fast-DDS@eae72fd caused the problem. I'll report a bug to upstream Fast-DDS. |
Thanks for checking it! You're right, the newest change was |
@clalancette, just checking, did you report the issue to FastDDS? |
Sorry, I forgot. I've done it now: eProsima/Fast-DDS#4230 |
In build log, we can see an error like this:
The list of paths we've seen is:
..\..\src\ros2\rosbag2\rosbag2_py\test\test_sequential_reader.py
(Windows Debug and Release)..\..\src\ros2\rosbag2\rosbag2_py\test\test_sequential_reader_multiple_files.py
(Windows Debug and Release)..\..\src\ros2\rosbag2\rosbag2_py\test\test_sequential_writer.py
(Windows Debug and Release)..\..\src\ros2\rosbag2\rosbag2_py\test\test_convert.py
(Windows Debug and Release)Jobs:
The text was updated successfully, but these errors were encountered: