Skip to content

Commit 6d94dfb

Browse files
authored
Fix: Resolve race condition in MoveGroupSequenceAction (#3125)
1 parent 6487249 commit 6d94dfb

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

moveit_planners/pilz_industrial_motion_planner/src/move_group_sequence_action.cpp

+3-1
Original file line numberDiff line numberDiff line change
@@ -72,8 +72,10 @@ void MoveGroupSequenceAction::initialize()
7272
{
7373
// start the move action server
7474
RCLCPP_INFO_STREAM(getLogger(), "initialize move group sequence action");
75+
// Use MutuallyExclusiveCallbackGroup to prevent race conditions in callbacks.
76+
// See: https://github.com/moveit/moveit2/issues/3117 for details.
7577
action_callback_group_ =
76-
context_->moveit_cpp_->getNode()->create_callback_group(rclcpp::CallbackGroupType::Reentrant);
78+
context_->moveit_cpp_->getNode()->create_callback_group(rclcpp::CallbackGroupType::MutuallyExclusive);
7779
move_action_server_ = rclcpp_action::create_server<moveit_msgs::action::MoveGroupSequence>(
7880
context_->moveit_cpp_->getNode(), "sequence_move_group",
7981
[](const rclcpp_action::GoalUUID& /* unused */,

0 commit comments

Comments
 (0)