You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ti...@apache.org on 2017/03/23 20:33:51 UTC
mesos git commit: Fixed flags logging in Docker executor.
Repository: mesos
Updated Branches:
refs/heads/master b2279c500 -> 47e04a4fc
Fixed flags logging in Docker executor.
Review: https://reviews.apache.org/r/57889/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/47e04a4f
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/47e04a4f
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/47e04a4f
Branch: refs/heads/master
Commit: 47e04a4fc4b5d3b40616409a26003e9ec4596ec1
Parents: b2279c5
Author: Till Toenshoff <to...@me.com>
Authored: Thu Mar 23 21:33:27 2017 +0100
Committer: Till Toenshoff <to...@me.com>
Committed: Thu Mar 23 21:33:27 2017 +0100
----------------------------------------------------------------------
src/docker/executor.cpp | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/47e04a4f/src/docker/executor.cpp
----------------------------------------------------------------------
diff --git a/src/docker/executor.cpp b/src/docker/executor.cpp
index b652a99..e7805d8 100644
--- a/src/docker/executor.cpp
+++ b/src/docker/executor.cpp
@@ -724,8 +724,6 @@ int main(int argc, char** argv)
return EXIT_FAILURE;
}
- std::cout << stringify(flags) << std::endl;
-
mesos::internal::logging::initialize(argv[0], flags, true); // Catch signals.
// Log any flag warnings (after logging is initialized).
@@ -733,13 +731,13 @@ int main(int argc, char** argv)
LOG(WARNING) << warning.message;
}
+ VLOG(1) << stringify(flags);
+
if (flags.help) {
cout << flags.usage() << endl;
return EXIT_SUCCESS;
}
- std::cout << stringify(flags) << std::endl;
-
if (flags.docker.isNone()) {
cerr << flags.usage("Missing required option --docker") << endl;
return EXIT_FAILURE;