Skip to content

Set envars to run tests with rmw_zenoh_cpp with multicast discovery#2776

Merged
ahcorde merged 2 commits intorollingfrom
ahcorde/rolling/test_zenoh_multicast
Mar 26, 2025
Merged

Set envars to run tests with rmw_zenoh_cpp with multicast discovery#2776
ahcorde merged 2 commits intorollingfrom
ahcorde/rolling/test_zenoh_multicast

Conversation

@ahcorde
Copy link
Copy Markdown
Contributor

@ahcorde ahcorde commented Mar 21, 2025

See ros2/rmw_zenoh#567

To test this PR, we can run the usual CI jobs to show there are no regressions and additionally run a CI job with a copy of the ros2.repos file from ros2/ros2#1649 with CI Scripts branch set to yadu/cargo

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from Yadunund March 21, 2025 18:20
@ahcorde ahcorde self-assigned this Mar 21, 2025
@Yadunund Yadunund marked this pull request as ready for review March 25, 2025 20:17
Signed-off-by: Yadunund <yadunund@gmail.com>
@Yadunund
Copy link
Copy Markdown
Member

Gist (default ros2.repos with changes from this branch): https://gist.githubusercontent.com/Yadunund/c4074b12ea5bdedcddae7b60f60c34bc/raw/7c4d586257be1804ee83101cd6f326d11b65b758/ros2_ci.repos
BUILD args: --packages-up-to rclcpp
TEST args: --packages-select rclcpp

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

Gist (with rmw_zenoh and changes from this branch) : https://raw.githubusercontent.com/ros2/ros2/3f873e9c5a79ce3b47f70199708b21c07007cd7d/ros2.repos
BUILD args: --packages-up-to rclcpp
TEST args: --packages-select rclcpp

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

@ros-discourse
Copy link
Copy Markdown

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/ros-pmc-minutes-for-march-25-2025/42807/1

@ahcorde ahcorde merged commit aae375b into rolling Mar 26, 2025
2 checks passed
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.

4 participants