Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

vmarmol
Copy link
Contributor

@vmarmol vmarmol commented May 22, 2015

Depends on #8586

No limits today, allows monitoring (and limiting in the future) the resource usage of other system services on the machine. From my tests I show ~1% of a core is used by these.

This PR only enables it for Debian (similar to #8586), although systemd does something very similar with /system.slice such that there are no rogue processes in root.

/cc @rjnagal @dchen1107

Running e2e now, but manual tests have been successful.

@rjnagal
Copy link
Contributor

rjnagal commented May 22, 2015

1% number is for 30 pods?

@vmarmol
Copy link
Contributor Author

vmarmol commented May 22, 2015

From my experiments, it does not depend on the number of pods. It seems to be some constant work done by the system. In GCE it is mainly due to network-related services it runs.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a comment that rolling back the flag requires a reboot.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@vmarmol vmarmol force-pushed the system-container branch from df9ff16 to 36d19e2 Compare May 22, 2015 17:14
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed offline: We should tie this to CgroupRoot.

@vmarmol vmarmol force-pushed the system-container branch from 36d19e2 to 16d1257 Compare May 22, 2015 18:02
@vmarmol
Copy link
Contributor Author

vmarmol commented May 22, 2015

e2e finished and it's green!

@rjnagal
Copy link
Contributor

rjnagal commented May 22, 2015

LGTM

@rjnagal rjnagal added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 22, 2015
@vmarmol
Copy link
Contributor Author

vmarmol commented May 22, 2015

Note to oncall, this depends on #8586

@dchen1107 dchen1107 self-assigned this May 22, 2015
@vmarmol vmarmol force-pushed the system-container branch from 16d1257 to 8b6d625 Compare May 22, 2015 18:19
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@googlebot
Copy link

CLAs look good, thanks!

vmarmol added 3 commits May 26, 2015 20:49
The system container is a resource-only container which contains all
non-kernel processes that are not already part of a container. This will
allow monitoring of their resource usage and limiting it (eventually).
@rjnagal
Copy link
Contributor

rjnagal commented May 29, 2015

@dchen1107 waiting for your nod for the freeze as you were looking at this one too.

@dchen1107
Copy link
Member

@vmarmol talked about this offline, I had some concerns. But on another side, I am really eager to have the entire system's resource usages monitored by kubernetes, and the stats could be propagated to upper layers, such as scheduler, monitoring, etc. to make smarter decision.

LGTM.

rjnagal added a commit that referenced this pull request May 29, 2015
@rjnagal rjnagal merged commit 50b9d62 into kubernetes:master May 29, 2015
@vmarmol vmarmol deleted the system-container branch May 29, 2015 17:08
@dchen1107 dchen1107 mentioned this pull request Jun 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants