mvukov / rules_ros2

Build ROS 2 with Bazel
Apache License 2.0
85 stars 46 forks source link

Fix flaky //ros2/test/diagnostics:diagnostic_heartbeat_test #195

Closed mvukov closed 1 year ago

ahans commented 1 year ago

@mvukov are you already on this? If not, I can have a look. I guess we could fix this by using reliable communication and replacing the recorder by a subscriber node in the test.

mvukov commented 1 year ago

Hi Alex, I saw a couple of failures of this test recently and wanted to make a note here. If you have some time, please go ahead. I do the same: replace the recorder with a subscriber.