You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ji...@apache.org on 2016/12/13 01:09:58 UTC
[5/5] mesos git commit: Added a TODO about agent subsystems check.
Added a TODO about agent subsystems check.
Make sure the corresponding cgroup isolator is enabled so that the
contaienr's processes will be moved out of the agent's cgroup.
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/0a96f5b5
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/0a96f5b5
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/0a96f5b5
Branch: refs/heads/master
Commit: 0a96f5b5b07692842092f0a64365c9fe4c890763
Parents: 354bca1
Author: Jie Yu <yu...@gmail.com>
Authored: Mon Dec 12 16:15:38 2016 -0800
Committer: Jie Yu <yu...@gmail.com>
Committed: Mon Dec 12 16:23:09 2016 -0800
----------------------------------------------------------------------
src/slave/slave.cpp | 5 +++++
1 file changed, 5 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/0a96f5b5/src/slave/slave.cpp
----------------------------------------------------------------------
diff --git a/src/slave/slave.cpp b/src/slave/slave.cpp
index 6553c29..673f44b 100644
--- a/src/slave/slave.cpp
+++ b/src/slave/slave.cpp
@@ -221,9 +221,14 @@ void Slave::initialize()
#ifdef __linux__
// Move the slave into its own cgroup for each of the specified
// subsystems.
+ //
// NOTE: Any subsystem configuration is inherited from the mesos
// root cgroup for that subsystem, e.g., by default the memory
// cgroup will be unlimited.
+ //
+ // TODO(jieyu): Make sure the corresponding cgroup isolator is
+ // enabled so that the container processes are moved to different
+ // cgroups than the agent cgroup.
if (flags.agent_subsystems.isSome()) {
foreach (const string& subsystem,
strings::tokenize(flags.agent_subsystems.get(), ",")) {