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(), ",")) {