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

Update jsk_travis to 0.5.6 #1171

Merged
merged 1 commit into from
Nov 19, 2019
Merged

Conversation

YutoUchimi
Copy link
Contributor

No description provided.

@YutoUchimi YutoUchimi added this to the 1.1.1 milestone Nov 12, 2019
@knorth55
Copy link
Member

knorth55 commented Nov 15, 2019

@YutoUchimi do you know the reason why current Travis CI build is failing?

@pazeshun
Copy link
Contributor

It seems that current indigo test (ROS_DISTRO=indigo USE_DEB=false EXTRA_DEB="ros-indigo-convex-decomposition ros-indigo-ivcon") takes too much time.

I compared results of that test in this PR and #1173.

This PR #1173
Whole test time 48 min 37 sec More than 50 min (Timeout)
rosdep_install 16 min 40 sec 16 min 51 sec
catkin_build 13 min 44 sec 14 min 15 sec
catkin_run_tests 10 min 52 sec ? (Timeout)

We can do...

@YutoUchimi
Copy link
Contributor Author

The main purpose of this PR is to upload cache correctly.
jsk_travis 0.5.5 and upper exits from docker container successfully, so caches specified in .travis.yml will be uploaded to server.
You can check uploaded cache of this repository by: https://travis-ci.org/jsk-ros-pkg/jsk_robot/caches?utm_medium=notification&utm_source=github_status

@pazeshun
Copy link
Contributor

pazeshun commented Nov 15, 2019

In catkin_run_tests...

Finished <<< aeroeus                                   [ 0.1 seconds ]
Finished <<< fetcheus                                  [ 1 minute and 42.6 seconds ]
Finished <<< jsk_baxter_web                            [ 0.2 seconds ]
Finished <<< jsk_fetch_diagnosis                       [ 0.1 seconds ]
Finished <<< jsk_pr2_calibration                       [ 0.1 seconds ]
Finished <<< jsk_robot_startup                         [ 0.2 seconds ]
Finished <<< jsk_robot_utils                           [ 18.6 seconds ]
Finished <<< naoqieus                                  [ 8.7 seconds ]
Finished <<< pr2_base_trajectory_action                [ 5.7 seconds ]
Finished <<< roseus_remote                             [ 0.2 seconds ]
Finished <<< jsk_baxter_startup                        [ 0.1 seconds ]
Finished <<< jsk_aero_startup                          [ 1.5 seconds ]
Finished <<< jsk_fetch_startup                         [ 5.2 seconds ]
Finished <<< jsk_pepper_startup                        [ 1.5 seconds ]
Finished <<< jsk_nao_startup                           [ 0.3 seconds ]
Finished <<< naoeus                                    [ 4.6 seconds ]
Finished <<< peppereus                                 [ 0.2 seconds ]
Finished <<< jsk_pr2_startup                           [ 0.2 seconds ]
Finished <<< baxtereus                                 [ 8 minutes and 20.0 seconds ]

baxtereus seems too heavy... I'll find what we can reduce.
-> baxter-moveit-nogui.test takes about 4 min and test-baxter.test takes about 4 min. failed for normal ik, try to move arms very slowly makes euslisp test slow, but IK fails without that.

@pazeshun
Copy link
Contributor

Building some packages in jsk_3rdparty is slow:

Finished <<< downward                                  [ 5 minutes and 40.7 seconds ]
Finished <<< assimp_devel                              [ 7 minutes and 14.2 seconds ]

pazeshun added a commit to pazeshun/jsk_robot that referenced this pull request Nov 18, 2019
With jsk-ros-pkg#1017, whole jsk_3rdparty is included in workspace on travis for jsk_fetch_startup, but we need respeaker_ros only
Fix jsk-ros-pkg#1171 (comment)
pazeshun added a commit to pazeshun/jsk_robot that referenced this pull request Nov 18, 2019
With jsk-ros-pkg#1017, whole jsk_3rdparty is included in workspace on travis for jsk_fetch_startup, but we need respeaker_ros only
See jsk-ros-pkg#1171 (comment)
pazeshun added a commit to pazeshun/jsk_robot that referenced this pull request Nov 18, 2019
With jsk-ros-pkg#1017, whole jsk_3rdparty is included in workspace on travis for jsk_fetch_startup, but we need respeaker_ros only
Also see jsk-ros-pkg#1171 (comment)
@pazeshun
Copy link
Contributor

@YutoUchimi @knorth55 I created #1177 and #1179 to reduce travis test time.
Both seem effective.

@k-okada k-okada merged commit 9af061a into jsk-ros-pkg:master Nov 19, 2019
@YutoUchimi YutoUchimi deleted the jsk_travis_0.5.6 branch November 19, 2019 09:14
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.

4 participants