[jsk_mbzirc] fix for travis test

start-jsk / jsk_mbzirc

4 stars 15 forks source link

[jsk_mbzirc] fix for travis test #70

Closed furushchev closed 8 years ago

furushchev commented 8 years ago

This PR includes some updates for passing build and test script on travis using catkin tools.

enable xvfb for gazebo rendering
pin version of catkin_tools to 0.3.0
suppress logging of apt-get
suppress logging of catkin
specify meta package name for testing to avoid including packages not in this repo for testing
source setup.bash after catkin build for testing
increase time limit to pass test on travis machine
add rostest / roslaunch to test_depend

furushchev commented 8 years ago

OK to merge.

furushchev commented 8 years ago

@k-okada Please merge if there is no problem. This fix is necessary for testing all other PRs.

k-okada commented 8 years ago

good work!

furushchev commented 8 years ago

@k-okada Thank you for merging!

k-okada commented 8 years ago

https://travis-ci.org/start-jsk/jsk_mbzirc/builds/127188062 error?

◉ Kei Okada

On Mon, May 2, 2016 at 7:52 PM, Furushchev notifications@github.com wrote:

@k-okada https://github.com/k-okada Thank you for merging!

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/start-jsk/jsk_mbzirc/pull/70#issuecomment-216197490

furushchev commented 8 years ago

@k-okada I also confirmed test sometimes fails. This may be because initial pose of ur arm on husky robot can be invalid, which let moveit client fails. I tried changing from time-limit 600 and retry 2 to time-limit 120 and retry 10 and travis succeeds 3 times without fail, is it ok to merge? (ref: https://github.com/start-jsk/jsk_mbzirc/pull/72 )