Skip to content

Set cgroup pids.max default systemd value for docker daemon#21297

Closed
clnperez wants to merge 1 commit intomoby:masterfrom
clnperez:systemd-tasksmax
Closed

Set cgroup pids.max default systemd value for docker daemon#21297
clnperez wants to merge 1 commit intomoby:masterfrom
clnperez:systemd-tasksmax

Conversation

@clnperez
Copy link
Copy Markdown
Contributor

- What I did
Updated the docker.service file for systemd
- How I did it
Typed
- How to verify it
cat /sys/fs/cgroup/pids/system.slice/docker.service/pids.max
Should be 1048576
- A picture of a cute animal (not mandatory but encouraged)

gc-stp-2

With the newer cgrouop PIDs limits in the 4.3 kernel, make sure
that the docker daemon isn't unreasonably limited.

Test failure with a basic docker command:
'docker run --rm -p 600:800 busybox /bin/sh'

See http://man7.org/linux/man-pages/man5/systemd.resource-control.5.html

    TasksMax=N
       "This controls the "pids.max" control group attribute."
       "Implies 'TasksAccounting=true'"

Signed-off-by: Christy Perez [email protected]

With the newer cgrouop PIDs limits in the 4.3 kernel, make sure
that the docker daemon isn't unreasonably limited.

Test failure with a basic docker command:
        'docker run --rm -p 600:800'

See http://man7.org/linux/man-pages/man5/systemd.resource-control.5.html

        TasksMax=N
           "This controls the "pids.max" control group attribute."
           "Implies 'TasksAccounting=true'"

Signed-off-by: Christy Perez <[email protected]>
@clnperez
Copy link
Copy Markdown
Contributor Author

Also, this should be changed for the other init-y things -- but I can't find the values and I know the systemd one. Apologies for being a bit lazy here. ;)

@calavera
Copy link
Copy Markdown
Contributor

/cc @jfrazelle, @cyphar.

@jessfraz
Copy link
Copy Markdown
Contributor

no we can't we already tried this it breaks older systed versions

@jessfraz
Copy link
Copy Markdown
Contributor

#20096

@jessfraz jessfraz closed this Mar 17, 2016
@clnperez
Copy link
Copy Markdown
Contributor Author

Aaah...duh. 😢 Thanks for the link, @jfrazelle.

@jessfraz
Copy link
Copy Markdown
Contributor

its super unfortunate :(

On Thu, Mar 17, 2016 at 1:41 PM, christyp [email protected] wrote:

Aaah...duh. [image: 😢] Thanks for the link, @jfrazelle
https://github.com/jfrazelle.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#21297 (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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants