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

Increase timeout waiting for server for ros2 client in tests #94

Merged
merged 1 commit into from
Oct 2, 2017

Conversation

dhood
Copy link
Member

@dhood dhood commented Oct 2, 2017

4s is not always enough, e.g.:
http://ci.ros2.org/view/packaging/job/packaging_linux-aarch64/195/testReport/junit/(root)/test_dynamic_bridge__rmw_fastrtps_cpp_RelWithDebInfo/test_dynamic_bridge_srv_1to2/:

(test_dynamic_bridge_srv_1to2__dynamic_bridge) pid 43970: ['/opt/ros/kinetic/env.sh', '/home/rosbuild/ci_scripts/ws/build/ros1_bridge/dynamic_bridge'] (stderr > stdout, all > console)
(test_dynamic_bridge_srv_1to2__ros1server) pid 43973: ['/opt/ros/kinetic/env.sh', '/home/rosbuild/ci_scripts/ws/build/ros1_bridge/test_ros1_server'] (stderr > stdout, all > console)
(test_dynamic_bridge_srv_1to2__ros2client) pid 43981: ['/home/rosbuild/ci_scripts/ws/build/ros1_bridge/test_ros2_client_cpp'] (stderr > stdout, all > console)
[test_dynamic_bridge_srv_1to2__ros1server] ?[31m[ERROR] [1506936168.263634670]: [registerPublisher] Failed to contact master at [localhost:11311].  Retrying...?[0m
[test_dynamic_bridge_srv_1to2__dynamic_bridge] ?[31m[ERROR] [1506936168.361192670]: [registerPublisher] Failed to contact master at [localhost:11311].  Retrying...?[0m
[test_dynamic_bridge_srv_1to2__ros2client] terminate called after throwing an instance of 'std::runtime_error'
[test_dynamic_bridge_srv_1to2__ros2client]   what():  Service is not available
(test_dynamic_bridge_srv_1to2__ros2client) rc -6

ci_packaging_linux: Build Status
ci_packaging_linux-aarch64: Build Status

@dhood dhood added the in review Waiting for review (Kanban column) label Oct 2, 2017
@dhood dhood self-assigned this Oct 2, 2017
Copy link
Member

@mikaelarguedas mikaelarguedas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@dhood dhood merged commit 71d8aee into master Oct 2, 2017
@dhood dhood deleted the increase_timeout branch October 2, 2017 21:53
@dhood dhood removed the in review Waiting for review (Kanban column) label Oct 2, 2017
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.

2 participants