52 lines
2.2 KiB
Diff
52 lines
2.2 KiB
Diff
|
From 419fd7449fe1a984f582731fcd4d9455000846b0 Mon Sep 17 00:00:00 2001
|
||
|
From: Alexander Morozov <lk4d4@docker.com>
|
||
|
Date: Wed, 4 Nov 2015 13:51:46 -0800
|
||
|
Subject: [PATCH] Use fs cgroups by default
|
||
|
|
||
|
Our implementation of systemd cgroups is mixture of systemd api and
|
||
|
plain filesystem api. It's hard to keep it up to date with systemd and
|
||
|
it already contains some nasty bugs with new versions. Ideally it should
|
||
|
be replaced with some daemon flag which will allow to set parent systemd
|
||
|
slice.
|
||
|
|
||
|
Signed-off-by: Alexander Morozov <lk4d4@docker.com>
|
||
|
---
|
||
|
daemon/execdriver/native/driver.go | 3 ---
|
||
|
docs/reference/commandline/daemon.md | 8 ++++----
|
||
|
2 files changed, 4 insertions(+), 7 deletions(-)
|
||
|
|
||
|
diff --git a/daemon/execdriver/native/driver.go b/daemon/execdriver/native/driver.go
|
||
|
index 09171c5..0b6cec3 100644
|
||
|
--- a/daemon/execdriver/native/driver.go
|
||
|
+++ b/daemon/execdriver/native/driver.go
|
||
|
@@ -74,9 +74,6 @@ func NewDriver(root, initPath string, options []string) (*Driver, error) {
|
||
|
// this makes sure there are no breaking changes to people
|
||
|
// who upgrade from versions without native.cgroupdriver opt
|
||
|
cgm := libcontainer.Cgroupfs
|
||
|
- if systemd.UseSystemd() {
|
||
|
- cgm = libcontainer.SystemdCgroups
|
||
|
- }
|
||
|
|
||
|
// parse the options
|
||
|
for _, option := range options {
|
||
|
diff --git a/docs/reference/commandline/daemon.md b/docs/reference/commandline/daemon.md
|
||
|
index 91fd3c6..0721538 100644
|
||
|
--- a/docs/reference/commandline/daemon.md
|
||
|
+++ b/docs/reference/commandline/daemon.md
|
||
|
@@ -452,11 +452,11 @@ single `native.cgroupdriver` option is available.
|
||
|
|
||
|
The `native.cgroupdriver` option specifies the management of the container's
|
||
|
cgroups. You can specify `cgroupfs` or `systemd`. If you specify `systemd` and
|
||
|
-it is not available, the system uses `cgroupfs`. By default, if no option is
|
||
|
-specified, the execdriver first tries `systemd` and falls back to `cgroupfs`.
|
||
|
-This example sets the execdriver to `cgroupfs`:
|
||
|
+it is not available, the system uses `cgroupfs`. If you omit the
|
||
|
+`native.cgroupdriver` option,` cgroupfs` is used.
|
||
|
+This example sets the `cgroupdriver` to `systemd`:
|
||
|
|
||
|
- $ sudo docker daemon --exec-opt native.cgroupdriver=cgroupfs
|
||
|
+ $ sudo docker daemon --exec-opt native.cgroupdriver=systemd
|
||
|
|
||
|
Setting this option applies to all containers the daemon launches.
|
||
|
|