CI:

• Linux
• Linux-aarch64
• Linux-rhel
• Windows

@clalancette @MiguelCompany can you review this?

@fujitatomoya the rpr job failed because ros2/rmw_fastrtps#781 was not yet merged.
I don't know whether that matters given the CI run in #2638 (comment), but I suppose that just pushing the changes again would retrigger it.

@MiguelCompany thanks for the review.

the rpr job failed because ros2/rmw_fastrtps#781 was not yet merged.

i believe as long as CI is green, that is fine. I am not sure how often rpr system is rebased...

i would like to check with rmw_zenoh before pushing this. @Yadunund is working on that, so do not want to bring another breaking trouble for them with this.

@clalancette can you review and merge this?

CC: @Yadunund i did confirm this test is passing with rmw_zenoh, see https://github.com/ros2/rclcpp/pull/2638/files#r1792386899

Pulls: #2638
Gist: https://gist.githubusercontent.com/fujitatomoya/6cc38769c5a97b7487be22039852e314/raw/657a433d4394c661f09019fb81e4e418a8b00ab1/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14685

• Linux
• Linux-aarch64
• Linux-rhel
• Windows

@ros-pull-request-builder retest this please

This is only a change in the rclcpp tests, so the Windows test failure must be unrelated. Going ahead and merging this one.