Skip to content

Commit

Permalink
dev: Stop testing ROS2 in parallel
Browse files Browse the repository at this point in the history
* It causes issues with multiple SITL ports and DDS topics overlapping

Signed-off-by: Ryan Friedman <[email protected]>
  • Loading branch information
Ryanf55 committed Sep 6, 2024
1 parent 05b438c commit 17b26e9
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions dev/source/docs/ros2.rst
Original file line number Diff line number Diff line change
Expand Up @@ -95,15 +95,20 @@ If the build fails, when you request help, please re-run the build in verbose mo
colcon build --packages-up-to ardupilot_dds_tests --event-handlers=console_cohesion+
If you'd like to test your installation, run:
If you'd like to test your ArduPilot ROS 2 installation, run:

.. code-block:: bash
cd ~/ros2_ws
source ./install/setup.bash
colcon test --packages-select ardupilot_dds_tests
colcon test --executor sequential --parallel-workers 0 --base-paths src/ardupilot --event-handlers=console_cohesion+
colcon test-result --all --verbose
The ``base-paths`` is used to limit testing only to ArduPilot.
We use a sequential executor and no parallel works because otherwise the tests try
to open multiple SITL processes on the same port, and the DDS traffic has cross-talk between
parallel tests.


Installation (MacOS)
====================
Expand Down

0 comments on commit 17b26e9

Please sign in to comment.