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

DockerClient.list should be able to figure out which Docker image and ports are associated with each container #207

Closed
wallrj opened this issue Jul 7, 2014 · 5 comments · Fixed by #878

Comments

@wallrj
Copy link
Contributor

wallrj commented Jul 7, 2014

We're not populating this information yet. Originally we were using geard so this was an issue, but with Docker directly it's easy.

@wallrj
Copy link
Contributor Author

wallrj commented Jul 7, 2014

See openshift/geard#229

@itamarst
Copy link
Contributor

itamarst commented Jul 7, 2014

This is not necessary for 0.1 since we're not going to be addressing changes to the application config.

@itamarst itamarst added this to the Release 0.3 milestone Sep 17, 2014
@itamarst itamarst changed the title GearClient.list should be able to figure out which Docker image is associated with each Gear unit DockerClient.list should be able to figure out which Docker image is associated with each container Sep 17, 2014
@itamarst itamarst changed the title DockerClient.list should be able to figure out which Docker image is associated with each container DockerClient.list should be able to figure out which Docker image and ports are associated with each container Sep 17, 2014
@itamarst itamarst added ready and removed ready labels Sep 24, 2014
@dwgebler dwgebler self-assigned this Oct 1, 2014
@dwgebler
Copy link
Contributor

dwgebler commented Oct 1, 2014

@robhaswell @exarkun @itamarst
Can this be labelled ready? I can't see any obvious reason for this issue to be blocked now.

@itamarst
Copy link
Contributor

itamarst commented Oct 1, 2014

One thing this branch should include is expanding the IDockerClient tests to cover adding ports and images.

@dwgebler
Copy link
Contributor

dwgebler commented Oct 6, 2014

I'll estimate this at 1.5 days and see what happens....

@dwgebler dwgebler removed the review label Oct 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants