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

DockerComposeContainer not working for version 2 of compose format #146

Closed
jlordiales opened this issue May 20, 2016 · 4 comments
Closed

Comments

@jlordiales
Copy link

Hi,
The DockerComposeContainer functionality seems to be broken if you are using the version 2 format of docker-compose where a new default network is created for each app.

The failing point seems to be the ambassador container, which fails to start with: "Error response from daemon: Cannot link to /5gcpep_api_1, as it does not belong to the default network".

Sample compose file:

version: '2'
services:
  db:
    image: postgres
    environment:
        POSTGRES_PASSWORD: test
        POSTGRES_USER: test
  api:
    build: .
    depends_on:
      - db
rnorth added a commit that referenced this issue May 21, 2016
…ogs, and eventually docker-compose v2 format. Refs #146, #147
@rnorth
Copy link
Member

rnorth commented May 21, 2016

I think this is going to necessitate an upgrade to v3 of the docker-java library to benefit from docker-java/docker-java#425.

@rnorth
Copy link
Member

rnorth commented Jun 5, 2016

Branch refactoring work continues on this - the issue with compose v2 networks remains unfortunately. I believe the necessary network API support went into docker-java in the last few days, so I hope it will appear in RC6 of that library.

@jlordiales
Copy link
Author

Sounds awesome! Looking forward to it

rnorth added a commit that referenced this issue Jun 10, 2016
…ogs, and eventually docker-compose v2 format. Refs #146, #147
@rnorth
Copy link
Member

rnorth commented Jun 11, 2016

v2 networks should be working properly now in the branch for #156

@rnorth rnorth modified the milestone: 1.1.0 Jul 2, 2016
rnorth added a commit that referenced this issue Jul 3, 2016
…mpatibility issues

Refactor Docker compose support, to allow scaling, better output logs, and eventually docker-compose v2 format. Refs #146, #147

Start support for docker-compose v2 environments, putting ambassador containers onto the right network to be able to access the compose-launched containers.

Fix Netty conflicts by using Jedis for testing (simpler dependencies)
rnorth added a commit that referenced this issue Jul 3, 2016
…mpatibility issues

Refactor Docker compose support, to allow scaling, better output logs, and eventually docker-compose v2 format. Refs #146, #147

Start support for docker-compose v2 environments, putting ambassador containers onto the right network to be able to access the compose-launched containers.

Fix Netty conflicts by using Jedis for testing (simpler dependencies)
@rnorth rnorth modified the milestone: 1.1.0 Jul 5, 2016
@rnorth rnorth closed this as completed Jul 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants