Closed bmagyar closed 1 month ago
This pull request is in conflict. Could you fix it @bmagyar?
We tested this PR (adding back in line 228 in controller_manager/controller_manager/spawner.py) and we were able to run ros2_control over a FastDDS discovery server.
I think I've touched up everything immediately needed. @fmauch , @wittenator could you guys please give me another round of feedback?
This pull request is in conflict. Could you fix it @bmagyar?
As written in different comments I think this needs some points to be addressed. I've updated thinks on my fork for cherry-picking.
If you create a PR to fix/spawner-interrupt
, we could directly merge them in.
As written in different comments I think this needs some points to be addressed. I've updated thinks on my fork for cherry-picking.
If you create a PR to
fix/spawner-interrupt
, we could directly merge them in.
Done in #1666
Note for myself: squash + merge from a PR with merge commits into another PR is not a good idea.
Attention: Patch coverage is 2.12766%
with 46 lines
in your changes missing coverage. Please review.
Project coverage is 84.60%. Comparing base (
07fb4f3
) to head (921c106
). Report is 2 commits behind head on master.
Conflict resolved from https://github.com/ros-controls/ros2_control/pull/1483