Use rmw_test_fixture to isolate ros2cli tests (backport #1062)#1106
Use rmw_test_fixture to isolate ros2cli tests (backport #1062)#1106fujitatomoya merged 5 commits intokiltedfrom
Conversation
Signed-off-by: Scott K Logan <logans@cottsay.net> (cherry picked from commit 75e004a)
* Enable RMW isolation for ros2doctor.test_report. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> * ResetEnvironment on shutdown is missing. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> * Clean up isolated ros2 daemon process when tests complete. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> * Clean up isolated ros2 daemon process for ros2topic. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> * Clean up isolated ros2 daemon process for ros2action. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> * Clean up isolated ros2 daemon process for ros2doctor. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> * Clean up isolated ros2 daemon process for ros2lifecycle and ros2node. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> * Clean up isolated ros2 daemon process for ros2param. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> --------- Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
|
@fujitatomoya I'll do a couple of extra attempts before giving up on replicating it. Tricky part would be trying to reproduce it on ci.ros2.org since the job with the regression it's on build.ros2.org. I added to this PR the commit that you referenced, I'm not sure if I'm missing something from the commits that were in the middle of those two but I will give a check in a bit. |
|
Re-running with extra commit from #1098 backported |
|
I was able to replicate the problem locally using the git clone git@github.com:ros-infrastructure/ros_buildfarm.git
cd ros_buildfarm/scripts/ci/
./generate_ci_script.py https://raw.githubusercontent.com/ros2/ros_buildfarm_config/ros2/index.yaml kilted nightly-release ubuntu noble amd64 > ci_job_kilted_all.sh
sh ci_job_kilted_all.shI'm taking a look to see how to modify the generated script to get access to the docker environment running the tests, and troubleshoot what's left open when the tests are run. |
|
I think we're missing a backport for from Rolling to Kilted in a dependency. |
|
For reference, after logging into the docker image being generated and running the scripts locally, here's a list of the services that remain open before execution of this test: Details |
Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
|
#1086 Was missing in the backport, probably this should do the trick. It works locally, although it's failing with a different error of the rmw_zenoh router not being available. Will run some tests before asking for reviews here. Currently this PR backport 1062, 1086 and 1098 |
Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
|
Kpr job seems to be missing extra configurations for Zenoh, showing various messages with:
And all the rmw_zenoh tests failing. The failures that I was tackling were solved, so I will run CI for this PR on ci.ros2.org and ask for reviews. |
|
I think this will need at least backporting ros2/ament_cmake_ros#46, I will open a PR there with ament_ros_cmake backports and test CI after those are merged. |
Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
|
@ros-pull-request-builder retest this please |
|
Ah nice, this only needed a rebuild after the backport in ament_cmake_ros was released. Re-running full CI: cc: @Crola1702 @miguelgonrod as buildfarmers, @cottsay as author of original PRs being backported |
|
@Crola1702 Is there a way to verify that the reported problems in the CI job for Windows Kilted were happening in main before this PR? Otherwise I think this is ready to be merged |
cottsay
left a comment
There was a problem hiding this comment.
The Windows test failures are upstream of ros2cli and appear unrelated.
Description
Use the
rmw_test_isolationsystem for creating a communication sandbox for each test.Fixes # (issue)
Is this user-facing behavior change?
Yes,
ros2cli.helpers.get_rmw_additional_envwill no longer enable multicast for Zenoh.Did you use Generative AI?
No
This is an automatic backport of pull request #1062 done by [Mergify](https://mergify.com).