[2/3] mesos git commit: Removed redundant 'root' user set in containerizer::launch().

[ji...@apache.org]

Removed redundant 'root' user set in containerizer::launch().

The 'user' in launch command is ignored, so it is not necessary to
explicitly set 'root' user to 'CommandInfo' in the case of command
task.

Review: https://reviews.apache.org/r/57401/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/9c0bbf1a
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/9c0bbf1a
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/9c0bbf1a

Branch: refs/heads/1.2.x
Commit: 9c0bbf1ad6bf649323877776a25e09fde7ec04f7
Parents: f575c58
Author: Gilbert Song <so...@gmail.com>
Authored: Thu Mar 9 12:42:40 2017 -0800
Committer: Jie Yu <yu...@gmail.com>
Committed: Thu Mar 9 13:50:22 2017 -0800

----------------------------------------------------------------------
 src/slave/containerizer/mesos/containerizer.cpp | 7 -------
 1 file changed, 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/9c0bbf1a/src/slave/containerizer/mesos/containerizer.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/containerizer.cpp b/src/slave/containerizer/mesos/containerizer.cpp
index b001d02..a23a6fa 100644
--- a/src/slave/containerizer/mesos/containerizer.cpp
+++ b/src/slave/containerizer/mesos/containerizer.cpp
@@ -1011,13 +1011,6 @@ Future<bool> MesosContainerizerProcess::launch(
     if (taskInfo->has_container()) {
       ContainerInfo* containerInfo = containerConfig.mutable_container_info();
       containerInfo->CopyFrom(taskInfo->container());
-
-      if (taskInfo->container().mesos().has_image()) {
-        // For command tasks, we need to set the command executor user
-        // as root as it needs to perform chroot (even when
-        // switch_user is set to false).
-        containerConfig.mutable_command_info()->set_user("root");
-      }
     }
   } else {
     // Other cases.