-
Notifications
You must be signed in to change notification settings - Fork 111
--cgroup-parent is coming to Docker 1.6.0 #25
Comments
I need to test this, but I believe cgroup parent doesn't work as you would think with systemd. Hopefully I'm wrong. |
As 1.6 is released: any chance you've tested this? |
I'll test once Docker 1.6 lands in CoreOS alpha. I assume that will be in the next day or so. |
Someone has an example with --cgroup-parent and systemd? |
Sorry to report the --cgroup-parent does nothing but rename the scope on systemd. For systemd it's a useless feature. In Docker 1.7 it will hopefully change. More info here docker-archive/libcontainer#549 but it won't address what systemd-docker does. There is a separate PR in Docker 1.7 moby/moby#10427 that will actually make systemd-docker work much much better. Ironically it disables the use of the systemd cgroup manager but this allows us to launch Docker containers in systemd that doesn't involve manually changing cgroups. |
@ibuildthecloud thanks for the answer, i will check later when docker 1.7 is available. |
I have docker 1.6 installed docker --version but I don't see option of -cgroup-parent, can somebody help me with an example too. ocker --version A self-sufficient runtime for linux containers. Options: Commands: Run 'docker COMMAND --help' for more information on a command. |
docker run --help Usage: docker run [OPTIONS] IMAGE [COMMAND] [ARG...] Run a command in a new container -a, --attach=[] Attach to STDIN, STDOUT or STDERR |
Ping re: moby/moby#10427, which is now merged. How does this affect (the necessity of) systemd-docker? |
It doesn't make it not required but it will make it such that it will work almost perfectly with no cgroup hackery. But this means you have to disable systemd integration on a systemd system. |
Post 1.7 release I will update systemd-docker to use cgroup-parent combined with disabling systemd integration. |
@ibuildthecloud That sounds great. A pair of followup questions:
|
I am not @ibuildthecloud, but that simpler solution is simplistic, in that the container is still not supervised at all. The big command streaming the logs to stdout is now |
@ibuildthecloud Docker 1.7 has been out for a bit, still planning to update systemd-docker? |
@ibotty I use that |
@ibuildthecloud Any update on this issue? |
Seems like moby/moby#11428 was merged and will be available in Docker 1.6.0, which addresses moby/moby#6791. I'm curious how this impacts this project?
The text was updated successfully, but these errors were encountered: