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/04/10 21:56:39 UTC
mesos git commit: Fixed flags logging in Docker executor.
Repository: mesos
Updated Branches:
refs/heads/1.0.x 80f97a259 -> 17ac0240a
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/17ac0240
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/17ac0240
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/17ac0240
Branch: refs/heads/1.0.x
Commit: 17ac0240a6136703e5bcac593cc79524a3eda92f
Parents: 80f97a2
Author: Till Toenshoff <to...@me.com>
Authored: Mon Apr 10 23:07:01 2017 +0200
Committer: Till Toenshoff <to...@me.com>
Committed: Mon Apr 10 23:07:01 2017 +0200
----------------------------------------------------------------------
src/docker/executor.cpp | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/17ac0240/src/docker/executor.cpp
----------------------------------------------------------------------
diff --git a/src/docker/executor.cpp b/src/docker/executor.cpp
index f46881b..93631ec 100644
--- a/src/docker/executor.cpp
+++ b/src/docker/executor.cpp
@@ -705,8 +705,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).
@@ -714,13 +712,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;