Skip to content

Add test for transient-local delivery with unbounded uint8 messages (backport #3195)#3202

Open
mergify[bot] wants to merge 1 commit into
lyricalfrom
mergify/bp/lyrical/pr-3195
Open

Add test for transient-local delivery with unbounded uint8 messages (backport #3195)#3202
mergify[bot] wants to merge 1 commit into
lyricalfrom
mergify/bp/lyrical/pr-3195

Conversation

@mergify

@mergify mergify Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Description

Add regression coverage for transient-local delivery of messages containing an unbounded uint8[].

The new test extends the existing wait_for_message test executable using test_msgs::msg::UnboundedSequences. It publishes a message before creating the subscription and verifies that the late-joining transient-local subscription receives the retained sample.

This test is specific for testing the case raised by ros2/rmw_fastrtps#897

Is this user-facing behavior change?

No.

Did you use Generative AI?

Yes. OpenAI GPT-5.6 was used to draft the code for this PR.


This is an automatic backport of pull request #3195 done by Mergify.

…3195)

Signed-off-by: CY Chen <cyc@nvidia.com>
(cherry picked from commit 859b29c)
@mergify

mergify Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@nvcyc

nvcyc commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Pulls: ros2/rmw_fastrtps#901, #3202, ros2/rmw_zenoh#1011
Gist: https://gist.githubusercontent.com/nvcyc/c6b01877e30f88e4dfff9f84d06c2c82/raw/984c7714f41916e4159972baa6cbb9a4531413b6/ros2.repos
BUILD args: --continue-on-error --packages-above-and-dependencies rmw_fastrtps_cpp rclcpp rmw_zenoh_cpp
TEST args: --packages-above rmw_fastrtps_cpp rclcpp rmw_zenoh_cpp
ROS Distro: lyrical
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/19896

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants