Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 8 commits into from
Mar 26, 2025

Conversation

ahcorde
Copy link
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

@ahcorde ahcorde requested a review from Yadunund March 21, 2025 18:45
@ahcorde ahcorde changed the title Set envars to run tests with rmw_zenoh_cpp with multicast discovery test_rclcpp: Set envars to run tests with rmw_zenoh_cpp with multicast discovery Mar 21, 2025
@ahcorde ahcorde changed the title test_rclcpp: Set envars to run tests with rmw_zenoh_cpp with multicast discovery Set envars to run tests with rmw_zenoh_cpp with multicast discovery Mar 21, 2025
ahcorde added 2 commits March 21, 2025 19:50
Signed-off-by: Alejandro Hernandez Cordero <[email protected]>
Signed-off-by: Alejandro Hernandez Cordero <[email protected]>
@Yadunund Yadunund force-pushed the ahcorde/rolling/test_zenoh_multicast branch from a506aab to 610382e Compare March 21, 2025 20:49
@Yadunund Yadunund force-pushed the ahcorde/rolling/test_zenoh_multicast branch from 31af1c7 to 1a10d62 Compare March 22, 2025 03:24
Signed-off-by: Yadunund <[email protected]>
@ros-discourse
Copy link

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

Signed-off-by: Yadunund <[email protected]>
@Yadunund Yadunund force-pushed the ahcorde/rolling/test_zenoh_multicast branch from 2c917bb to 36b76fe Compare March 25, 2025 23:22
@Yadunund Yadunund marked this pull request as ready for review March 25, 2025 23:53
@Yadunund
Copy link
Member

Gist (default ros2.repos with changes from this branch): https://gist.githubusercontent.com/Yadunund/c4074b12ea5bdedcddae7b60f60c34bc/raw/a9d2a2fcb87a7f34426aacfd4836a0ee9c6ad6b5/ros2_ci.repos
BUILD_ARGS: --packages-up-to test_communication test_quality_of_service test_rclcpp
TEST_ARGS: --packages-select test_communication test_quality_of_service test_rclcpp

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

@ahcorde ahcorde merged commit e797b66 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