moveit / moveit2

:robot: MoveIt for ROS 2
https://moveit.ai/
BSD 3-Clause "New" or "Revised" License
1.03k stars 508 forks source link

CI: RDF loader test is flaky #1334

Closed henrygerardmoore closed 1 year ago

henrygerardmoore commented 2 years ago

Description

The RDF Loader appears to fail in CI for no reason on one commit and then succeeds on the other where only a comment was removed.

Your environment

Steps to reproduce

Only encountered this once, so it is difficult to say how to reproduce it. Possibly just repeated runs of CI will do it.

Expected behaviour

Test should pass consistently

Actual behaviour

Test appears to fail inconsistently

Backtrace or Console output

CI for failing test CI for passing test

mikeferguson commented 2 years ago

I notice in the logs "terminate called without an active exception" - which usually means that a std::thread went out of scope while not joined/stopped - I notice we have a std::thread in this test - but I don't actually see how it could go out of scope...

I fixed an instance of this in https://github.com/ros-planning/moveit2/pull/1384

henningkayser commented 1 year ago

I haven't seen this one lately. Are we still running into this? Maybe it really was an unrelated threading issue somewhere...

vatanaksoytezer commented 1 year ago

I think I saw this like a few months ago, but haven't seen lately.

henningkayser commented 1 year ago

closing for now