Fix goal updater for CI

[SteveMacenski]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines	Coverage Δ
...avior_tree/plugins/decorator/goal_updater_node.cpp	100.00% <100.00%> (+21.73%)	⬆️

... and 3 files with indirect coverage changes

[SteveMacenski]

@clalancette should I go ahead and merge this or do you want me to hold off for an rclcpp discussion?

[clalancette]

@clalancette should I go ahead and merge this or do you want me to hold off for an rclcpp discussion?

So I think what we should do here is to open an issue against rclcpp with all of the data we currently have (a self-contained reproducer would be ideal, but not required). Once that issue is open, I would then go ahead and merge this workaround for now.

@SteveMacenski Does that make sense to you? Do you mind doing that?

[SteveMacenski]

@clalancette done: ros2/rclcpp#2589

I would like to merge this however to get our CI green, but its easy enough to revert for testing the failure case as part of the development for a fix

[tonynajjar]

@SteveMacenski I noticed that this was not backported to Jazzy branch. I triggered it