-
Notifications
You must be signed in to change notification settings - Fork 7
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 CI and docker-compose issues #23
Comments
So this is why travis is failing on ome/openmicroscopy#6108? |
Does the suggested fix in the first post (add |
@manics Sorry, I'm not sure which of the links above you mean by the "first post". |
Sorry, I meant this post docker/compose#6316 (comment) Add |
I'm looking at a bunch of scripts in https://github.com/ome/omero-test-infra but I can't see any container names. Is it something like:
|
Remove hard-coding of `$project_$service_1` in favor of `docker ps` with appropriate filters. see: ome/omero-plugins#23
Proposal PR opened. |
ome/omero-test-infra#43 is now merged and, e.g., ome/omero-figure#342 went green. Closing and looking for jobs that need re-running. (Help appreciated there.) |
The cron-based Travis builds started to fail lately - see https://travis-ci.org/ome/omero-plugins/builds/595701347.
The issue is similar to the one discussed in https://travis-ci.community/t/problematic-docker-compose-version-1-23-1/5362:
xenial
rather thantrusty
(https://blog.travis-ci.com/2019-04-15-xenial-default-build-environment)xenial
environment ships docker-compose 1.23.1 (https://docs.travis-ci.com/user/reference/xenial/#docker)docker-compose
(https://github.com/docker/compose/blob/1.23.2/CHANGELOG.md)sbesson@c874909562fdc698167d5fbf082657d94ac42b31 is a potential workaround for solving the issue using the pre-installation approach mentioned in https://docs.travis-ci.com/user/docker/#using-docker-compose to install
docker-compose 1.23.2
which does not breakomero-test-infra
assumptions. A downside of this approach is that it needs to be opened against every repository consumingomero-test-infra
.Another approach would be to update
omero-test-infra
to compute the container ID without relying on the naming convention e.g. docker/compose#6316 (comment)The text was updated successfully, but these errors were encountered: