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

Docker's Makefile uses "docker" namespace #2

Closed
tianon opened this issue May 1, 2015 · 7 comments · Fixed by moby/moby#12936
Closed

Docker's Makefile uses "docker" namespace #2

tianon opened this issue May 1, 2015 · 7 comments · Fixed by moby/moby#12936

Comments

@tianon
Copy link
Member

tianon commented May 1, 2015

Docker's development Makefile currently uses the docker namespace for building development images: https://github.com/docker/docker/blob/3ea59f8991a4d70254582759044316e522a09ee9/Makefile#L29

We need to decide if this is a problem, and whether we should fix it by naming this image something else, or by sending a PR to Docker.

Especially wondering @jfrazelle's opinion on this one.

I've just barely reorganized this repo so that we can have docker:latest be just the Docker CLI, and docker:dind include all the hacky bits to do Docker-in-Docker, since they're pretty intrusive and require --privileged, and just being able to use the CLI from inside an image would definitely be useful.

@jessfraz
Copy link

jessfraz commented May 2, 2015

oh man hmmmm maybe dockerdev, docker-engine, naming things is so hard, what if we just did engine

@tianon
Copy link
Member Author

tianon commented May 2, 2015

For this image's name, or as a PR against Makefile to change the name people build Docker with?

@jessfraz
Copy link

jessfraz commented May 2, 2015

oh def our image should be called docker

but idk what to name the dev docker we build and test in ;)

@tianon
Copy link
Member Author

tianon commented May 2, 2015 via email

@jessfraz
Copy link

jessfraz commented May 2, 2015

ah oerfect :)

On Fri, May 1, 2015 at 5:20 PM, Tianon Gravi notifications@github.com
wrote:

Right, OK. 👍

"docker-dev" is an official image that has builds of Docker's own
Dockerfile, so maybe we should just use that same name? (especially since
it only has builds of the releases)


Reply to this email directly or view it on GitHub
#2 (comment).

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC 511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

@jessfraz
Copy link

jessfraz commented May 2, 2015

perfect*

On Fri, May 1, 2015 at 5:21 PM, Jessica Frazelle me@jessfraz.com wrote:

ah oerfect :)

On Fri, May 1, 2015 at 5:20 PM, Tianon Gravi notifications@github.com
wrote:

Right, OK. 👍

"docker-dev" is an official image that has builds of Docker's own
Dockerfile, so maybe we should just use that same name? (especially since
it only has builds of the releases)


Reply to this email directly or view it on GitHub
#2 (comment)
.

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC 511E 18F3 685C 0022 BFF3
pgp.mit.edu
http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC 511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

tianon added a commit to tianon/docker that referenced this issue May 2, 2015
…docker"

See docker-library/docker#2 for some of the context around this; essentially, we're looking to create an official `docker` image that includes the Docker CLI (and the dependencies necessary for easy Docker-in-Docker), but it makes sense to use the `docker` namespace for that image.  The `docker-dev` official image is already builds of Docker's own `Dockerfile` (but specifically of releases), so this seemed like a good fit. 👍

Signed-off-by: Andrew "Tianon" Page <admwiggin@gmail.com>
@tianon
Copy link
Member Author

tianon commented May 4, 2015

Fixed by moby/moby#12936 👍

@tianon tianon closed this as completed May 4, 2015
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 a pull request may close this issue.

2 participants