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.

Sign up for GitHub

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

Jump to bottom

[CI] Add ROS Melodic job. Add ABI check for K and M. #52

Merged
merged 2 commits into from
May 21, 2018
Merged

[CI] Add ROS Melodic job. Add ABI check for K and M. #52

merged 2 commits into from
May 21, 2018

Conversation

130s
Copy link
Member

@130s 130s commented May 21, 2018

No description provided.

@130s
Copy link
Member Author

130s commented May 21, 2018

Build process failed with exhaustive memory error. I've seen this in other places on Travis for ar_track_alvar. Also some jobs stalled for no obvious reason printed.

Decreasing concurrency.

https://travis-ci.org/ros-perception/ar_track_alvar/jobs/381526078#L603

Errors     << ar_track_alvar:make /root/catkin_ws/logs/ar_track_alvar/build.make.000.log
virtual memory exhausted: Cannot allocate memory
make[2]: *** [CMakeFiles/findMarkerBundles.dir/nodes/FindMarkerBundles.cpp.o] Error 1
make[1]: *** [CMakeFiles/findMarkerBundles.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make: *** [all] Error 2
cd /root/catkin_ws/build/ar_track_alvar; catkin build --get-env ar_track_alvar | catkin env -si  /usr/bin/make --jobserver-fds=6,7 -j; cd -
...............................................................................
Failed     << ar_track_alvar:make                  [ Exited with code 2 ]
Failed    <<< ar_track_alvar                       [ 2 minutes and 36.9 seconds ]
[build] Successful packages:
 [Successful] ar_track_alvar_msgs
[build] Failed packages:
 [    Failed] ar_track_alvar

@130s
[CI] Reduce concurrency. Allow-failure ROS Lunar job.
2d9247c 
- See ros-perception#52 (comment)
- ROS Lunar won't be the latest after Melodic release, so reduce importance.
130s added a commit to 130s/ar_track_alvar that referenced this pull request May 21, 2018
@130s
[CI] Reduce concurrency (see ros-perception#52 (comment)).
8cfd806
@130s 130s mentioned this pull request May 21, 2018
Merged
@130s
Copy link
Member Author

130s commented May 21, 2018

Travis CI passed. ROS buildfarm test failed in the same way as #39 (comment), which is ignorable (in the next update to rosdistro I'll disable that feature). Merging.

@130s 130s merged commit a870d5f into ros-perception:kinetic-devel May 21, 2018
@130s 130s deleted the ci_melodic/k branch May 21, 2018 10:11
130s added a commit to 130s/rosdistro that referenced this pull request May 21, 2018
@130s
[lunar] remove PR test from ar_track_alvar.
d46e451 
ros-perception/ar_track_alvar#52 (comment)
dirk-thomas pushed a commit to ros/rosdistro that referenced this pull request May 21, 2018
@130s @dirk-thomas
ar_track_alvar: 0.7.1-0 in 'melodic/distribution.yaml' [bloom] (#17918)
c381c89 
* ar_track_alvar: 0.7.1-0 in 'melodic/distribution.yaml' [bloom]

* [lunar] remove PR test from ar_track_alvar.
ros-perception/ar_track_alvar#52 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@130s