gazebosim / ros_gz

Integration between ROS (1 and 2) and Gazebo simulation
https://gazebosim.org
Apache License 2.0
261 stars 139 forks source link

Move gzserver logic to its action #646

Closed caguero closed 4 days ago

caguero commented 5 days ago

🦟 Bug fix

Summary

In a similar way of #628, this patch refactors the way we are launching gzserver. Now the logic lives in the action and all the launch files use this action.

How to test it?

You can try checking these launch files:

ros2 launch ros_gz_sim gz_server.launch.py world_sdf_file:=shapes.sdf use_composition:=True create_own_container:=True
ros2 launch ros_gz_sim gz_server.launch world_sdf_file:=shapes.sdf use_composition:=True create_own_container:=True
ros2 launch ros_gz_sim ros_gz_sim.launch.py bridge_name:=ros_gz_bridge config_file:=/home/caguero/ros_gz_ws/src/ros_gz/ros_gz_bridge/test/config/full.yaml world_sdf_file:=empty.sdf use_composition:=True create_own_container:=True
ros2 launch ros_gz_sim ros_gz_sim.launch bridge_name:=ros_gz_bridge config_file:=/home/caguero/ros_gz_ws/src/ros_gz/ros_gz_bridge/test/config/full.yaml world_sdf_file:=empty.sdf use_composition:=True create_own_container:=True

And check that there are no errors during launch.

Checklist

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

ahcorde commented 4 days ago

https://github.com/Mergifyio backport jazzy

mergify[bot] commented 4 days ago

backport jazzy

✅ Backports have been created

* [#649 Move gzserver logic to its action (backport #646)](https://github.com/gazebosim/ros_gz/pull/649) has been created for branch `jazzy`