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 CI and docker-compose issues #23

Closed
sbesson opened this issue Oct 11, 2019 · 7 comments
Closed

Travis CI and docker-compose issues #23

sbesson opened this issue Oct 11, 2019 · 7 comments

Comments

@sbesson
Copy link
Member

sbesson commented Oct 11, 2019

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:

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 break omero-test-infra assumptions. A downside of this approach is that it needs to be opened against every repository consuming omero-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)

@will-moore
Copy link
Member

So this is why travis is failing on ome/openmicroscopy#6108?
Definitely a blocker for me, so 👍 for any fix that works, thanks.

@manics
Copy link
Member

manics commented Oct 11, 2019

Does the suggested fix in the first post (add container_name in docker-compose.yml) work?

@will-moore
Copy link
Member

@manics Sorry, I'm not sure which of the links above you mean by the "first post".
What change do you need to make to which file?
https://github.com/ome/omero-test-infra/blob/master/docker-compose.yml ?

@manics
Copy link
Member

manics commented Oct 11, 2019

Sorry, I meant this post docker/compose#6316 (comment)

Add containername: <xxx> to all services in https://github.com/ome/omero-test-infra/blob/master/docker-compose.yml to match what the scripts expect

@will-moore
Copy link
Member

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:

    omero:
        container_name: omero-server

joshmoore added a commit to joshmoore/omero-test-infra that referenced this issue Oct 16, 2019
Remove hard-coding of `$project_$service_1` in favor
of `docker ps` with appropriate filters.

see: ome/omero-plugins#23
@joshmoore
Copy link
Member

Proposal PR opened.

@joshmoore
Copy link
Member

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.)

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

No branches or pull requests

4 participants