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/11/29 19:41:24 UTC
[5/9] mesos git commit: Moved a VLOG to a proper place in
MesosContainerizer.
Moved a VLOG to a proper place in MesosContainerizer.
We should print the VLOG after we calculate the launch flags.
Review: https://reviews.apache.org/r/54098
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/08ff8763
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/08ff8763
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/08ff8763
Branch: refs/heads/master
Commit: 08ff8763e5e43761e08b1297b16eed6fef4a4fc7
Parents: c06c697
Author: Jie Yu <yu...@gmail.com>
Authored: Thu Nov 24 22:41:03 2016 -0800
Committer: Jie Yu <yu...@gmail.com>
Committed: Tue Nov 29 11:40:50 2016 -0800
----------------------------------------------------------------------
src/slave/containerizer/mesos/containerizer.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/08ff8763/src/slave/containerizer/mesos/containerizer.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/containerizer.cpp b/src/slave/containerizer/mesos/containerizer.cpp
index 9b33495..72d55e7 100644
--- a/src/slave/containerizer/mesos/containerizer.cpp
+++ b/src/slave/containerizer/mesos/containerizer.cpp
@@ -1521,9 +1521,6 @@ Future<bool> MesosContainerizerProcess::_launch(
launchFlags.runtime_directory = runtimePath;
#endif // __WINDOWS__
- VLOG(1) << "Launching '" << MESOS_CONTAINERIZER << "' with flags '"
- << launchFlags << "'";
-
// For now we need to special case entering a parent container's
// mount namespace. We do this to ensure that we have access to
// the binary we launch with `launcher->fork()`.
@@ -1555,6 +1552,9 @@ Future<bool> MesosContainerizerProcess::_launch(
}
#endif // __linux__
+ VLOG(1) << "Launching '" << MESOS_CONTAINERIZER << "' with flags '"
+ << launchFlags << "'";
+
// Fork the child using launcher.
vector<string> argv(2);
argv[0] = MESOS_CONTAINERIZER;