moveit / moveit

:robot: The MoveIt motion planning framework
http://moveit.ros.org
BSD 3-Clause "New" or "Revised" License
1.69k stars 947 forks source link

CI broken #2696

Closed rhaschke closed 3 years ago

rhaschke commented 3 years ago

We still have a serious issue in our GHA config: While the CI checks for PR #2691 (as well as the first master build after merging it) succeeded:

now builds are failing due to the -pedantic flag: https://github.com/ros-planning/moveit/actions/runs/885022060

I think the workspace caching isn't correctly working yet: In the succeeding master build, the target_ws was restored from cache, in the failing one not (for a reason I don't understand).

I'm afraid we need to remove the -pedantic flag for now because we cannot quickly adapt prbt_ikfast_manipulator_plugin in moveit_resources. In any case, we should understand the errornous caching. @tylerjw, do you have any insight?

v4hn commented 3 years ago

I'm afraid we need to remove the -pedantic flag for now

Fair enough. Please go ahead I don't have the time right now. I will work on fixes later.

rhaschke commented 3 years ago

The caching issue is hard to grasp... Closed for now.