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 2014/06/18 23:56:59 UTC

[2/2] git commit: Updated mesos to use the new Subprocess interface with flags.

Updated mesos to use the new Subprocess interface with flags.

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


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

Branch: refs/heads/master
Commit: 02fa100eaf3d92261e39460b0aeb48c89306fc8a
Parents: 82eb111
Author: Jie Yu <yu...@gmail.com>
Authored: Wed Jun 18 12:45:37 2014 -0700
Committer: Jie Yu <yu...@gmail.com>
Committed: Wed Jun 18 14:56:22 2014 -0700

----------------------------------------------------------------------
 src/launcher/launcher.cpp                          | 1 +
 src/slave/containerizer/external_containerizer.cpp | 1 +
 src/slave/containerizer/mesos_containerizer.cpp    | 1 +
 src/tests/slave_tests.cpp                          | 1 +
 4 files changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/02fa100e/src/launcher/launcher.cpp
----------------------------------------------------------------------
diff --git a/src/launcher/launcher.cpp b/src/launcher/launcher.cpp
index 5585aad..872e2e8 100644
--- a/src/launcher/launcher.cpp
+++ b/src/launcher/launcher.cpp
@@ -211,6 +211,7 @@ process::Future<Option<int> > Operation::launch(
       Subprocess::PIPE(),
       Subprocess::PIPE(),
       Subprocess::PIPE(),
+      None(),
       environment);
 
   if (s.isError()) {

http://git-wip-us.apache.org/repos/asf/mesos/blob/02fa100e/src/slave/containerizer/external_containerizer.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/external_containerizer.cpp b/src/slave/containerizer/external_containerizer.cpp
index 923046c..96c434b 100644
--- a/src/slave/containerizer/external_containerizer.cpp
+++ b/src/slave/containerizer/external_containerizer.cpp
@@ -1119,6 +1119,7 @@ Try<Subprocess> ExternalContainerizerProcess::invoke(
       Subprocess::PIPE(),
       Subprocess::PIPE(),
       Subprocess::PIPE(),
+      None(),
       environment,
       lambda::bind(&setup, sandbox.isSome() ? sandbox.get().directory
                                             : string()));

http://git-wip-us.apache.org/repos/asf/mesos/blob/02fa100e/src/slave/containerizer/mesos_containerizer.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos_containerizer.cpp b/src/slave/containerizer/mesos_containerizer.cpp
index e5c159d..61c0a8d 100644
--- a/src/slave/containerizer/mesos_containerizer.cpp
+++ b/src/slave/containerizer/mesos_containerizer.cpp
@@ -663,6 +663,7 @@ Future<Nothing> MesosContainerizerProcess::fetch(
       Subprocess::PIPE(),
       Subprocess::PIPE(),
       Subprocess::PIPE(),
+      None(),
       environment);
 
   if (fetcher.isError()) {

http://git-wip-us.apache.org/repos/asf/mesos/blob/02fa100e/src/tests/slave_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/slave_tests.cpp b/src/tests/slave_tests.cpp
index 9dc4046..3a45191 100644
--- a/src/tests/slave_tests.cpp
+++ b/src/tests/slave_tests.cpp
@@ -351,6 +351,7 @@ TEST_F(SlaveTest, MesosExecutorWithOverride)
         process::Subprocess::PIPE(),
         process::Subprocess::PIPE(),
         process::Subprocess::PIPE(),
+        None(),
         environment);
 
   ASSERT_SOME(executor);