-
Notifications
You must be signed in to change notification settings - Fork 49
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
Get ci.ros2.org to pass with rmw_zenoh #567
Comments
Hi @Yadunund! Since the previous issue has been closed, let me answer you here. We can set
for port auto selection, but I found that more tests fail compared to generating the ports in a script. |
Regarding the failures reported by CI, I suspect the rmw_zenohd router doesn't work well. See the log here. |
@YuanYuYuan to clarify, all these tests are failing because the Zenoh router has not been started before running the tests. Hence we need to add changes like ros2/rcl#1218 to these packages. |
I see. Then let's focus on the multicast version first. Here are the tests that haven't steadily passed on a local Ubuntu.
|
Remaining test failures: https://ci.ros2.org/job/ci_windows/23718/#showFailuresLink
|
CI started with: https://raw.githubusercontent.com/ros2/ros2/3f873e9c5a79ce3b47f70199708b21c07007cd7d/ros2.repos
|
After a huge effort in #493, we have all tests for all packages in the ros2.repos file passing with rmw_zenoh in local workspaces 🎉
However this required us to start the zenohd router locally which does not happen automatically when running a CI job on ci.ros2.org. The ros-infrastructure team is working on middleware agnostic mechanisms to start a pre-defined process but this may not be ready before the RMW freeze for Kilted on April 7.
A workaround in the meanwhile (to demonstrate that rmw_zenoh is indeed capable of passing CI) is to have ci.ros2.org run the usual tests with multicast scouting enabled for discovery. This would be akin to DDS-based middlewares. Once the infra tooling is in place, we can revert these changes.
The good news is that while we already have all tests for all packages passing locally, when a job runs on ci.ros2.org, tests for most packages run with the default middleware and only a select few in the ros2.repos file generate tests for every middleware found in the workspace. These packages rely on the
get_available_rmw_implementations
macro to retrieve all middlewares and generate tests for these middlewares. Therefore the goal is to update the tests in these repos to set environment variables to run rmw_zenoh tests with multicast discovery enabled. ie, setThe packages that require changes are
For detailed failures see
The text was updated successfully, but these errors were encountered: