ros / dynamic_reconfigure

BSD 3-Clause "New" or "Revised" License
48 stars 111 forks source link

rostest times out on Debian Stretch and recent ubuntus #103

Closed mikaelarguedas closed 5 years ago

mikaelarguedas commented 6 years ago

This seem to be Ubuntu Distro related and not ROS Distro related, consistently fails on Debian Stretch, Ubuntu Artful and Bionic. Example jobs with hanging test: http://build.ros.org/view/Mdev/job/Mdev__dynamic_reconfigure__ubuntu_bionic_amd64/1/ http://build.ros.org/view/Mdev_ds/job/Mdev_ds__dynamic_reconfigure__debian_stretch_amd64/1/

krixkrix commented 5 years ago

On debian stretch this test fails for me as well:

rostest dynamic_reconfigure test_cpp_simple_client.launch

Or to be accurate: all tests passes but the test itself times out.

If I change this line in test_client.cpp:

ros::AsyncSpinner spinner(2);

To:

ros::AsyncSpinner spinner(1);

...then the tests succeeds.

mikaelarguedas commented 5 years ago

thanks @krixkrix for looking into it!

I opened https://github.com/ros/dynamic_reconfigure/pull/119 that should address the test failure without modifying the number of threads used by the spinner.