ros2 / rosbag2

Apache License 2.0
285 stars 251 forks source link

[jazzy] Add cli option compression-threads-priority (backport #1768) #1778

Closed mergify[bot] closed 3 months ago

mergify[bot] commented 3 months ago

This PR adds --compression-threads-priority CLI parameter for the ros2 bag record command to be able to set up compression threads priority.

MichaelOrlov commented 3 months ago

Note: There are no API/ABI breaking changes. Only ABI breaking changes in test, but is is ok since we don't guarantee API/ABI compatibility for tests. It is ok to merge it on jazzy.

MichaelOrlov commented 3 months ago

Rebased on top latest Jazzy to get fixes for CI failures.

MichaelOrlov commented 3 months ago

Pulls: ros2/rosbag2#1778 Gist: https://gist.githubusercontent.com/MichaelOrlov/2a50f6b2f7084f911fa6be2df4eaa331/raw/8a1afd39f446a9fd9e27eea7559af49ba0f19815/ros2.repos BUILD args: --packages-above-and-dependencies rosbag2_compression ros2bag rosbag2_transport rosbag2_py TEST args: --packages-above rosbag2_compression ros2bag rosbag2_transport rosbag2_py ROS Distro: jazzy Job: ci_launcher ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14399

MichaelOrlov commented 3 months ago

@ros-pull-request-builder retest this please