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

.travis.yml: switch to orocos/ros2-ci Docker images without preinstalled rtt_ros2_integration for CI #7

Merged
merged 1 commit into from
Jul 29, 2020

Conversation

meyerj
Copy link
Member

@meyerj meyerj commented Jul 29, 2020

Same as orocos/rtt_ros2_integration#26 for rtt_ros2_integration:

Once orocos/orocos-docker-images#3 has been merged and the issues with automated builds on Docker Hub have been solved, the orocos/ros2 images will have rtt_ros2_integration and rtt_ros2_common_interfaces preinstalled. This might be bad for CI testing of those two repos, if older versions of the packages to be tested are preinstalled in an underlay.

Therefore a new Docker repository orocos/ros2-ci has been created, whose images are equivalent to the ones in orocos/ros2 (after orocos/orocos-docker-images#3) but stop at the orocos_toolchain target and hence only have the Orocos Toolchain packages preinstalled.

A fundamental difference compared to the previous images is that the Orocos Toolchain and ROS integration packages are installed to /opt/orocos/${ROS_DISTRO} now and not to /opt/ros/${ROS_DISTRO}. The main purpose of this pull request is to verify whether industrial_ci can deal with that. industrial_ci/src/tests/source_tests.sh only extends /opt/ros/${ROS_DISTRO} and there is no configuration variable to change that, but /opt/orocos/${ROS_DISTRO} is already sourced in the orocos_entrypoint.sh script and hence the required variables should be already set (CMAKE_PREFIX_PATH, LD_LIBRARY_PATH, OROCOS_TARGET, ...).

The current set of images in orocos/ros2-ci have been built and pushed manually because automated builds for orocos/orocos-docker-images#3 on Docker Hub still fail, likely because of resource constraints.

@meyerj meyerj merged commit daa3a8b into master Jul 29, 2020
@meyerj meyerj deleted the feature/travis-switch-to-ros2-ci-docker-images branch July 29, 2020 10:40
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.

1 participant