Raised the logging level of some check and health check messages. Some users pointed out that always logging the result of checks and health checks makes it easier to debug problems.
Review: https://reviews.apache.org/r/61791/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/8347ec09 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/8347ec09 Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/8347ec09 Branch: refs/heads/master Commit: 8347ec09f15989b822f48c000043c6547c25b5f4 Parents: 6d778fa Author: Gastón Kleiman <gas...@mesosphere.io> Authored: Mon Aug 21 15:28:39 2017 -0700 Committer: Greg Mann <gregorywm...@gmail.com> Committed: Mon Aug 21 15:43:29 2017 -0700 ---------------------------------------------------------------------- src/checks/checker_process.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/8347ec09/src/checks/checker_process.cpp ---------------------------------------------------------------------- diff --git a/src/checks/checker_process.cpp b/src/checks/checker_process.cpp index 657e6c0..4aa5b2d 100644 --- a/src/checks/checker_process.cpp +++ b/src/checks/checker_process.cpp @@ -797,7 +797,7 @@ void CheckerProcess::processCommandCheckResult( // see MESOS-7242. if (future.isReady() && WIFEXITED(future.get())) { const int exitCode = WEXITSTATUS(future.get()); - VLOG(1) << name << " for task '" << taskId << "' returned: " << exitCode; + LOG(INFO) << name << " for task '" << taskId << "' returned: " << exitCode; CheckStatusInfo checkStatusInfo; checkStatusInfo.set_type(check.type()); @@ -958,8 +958,8 @@ void CheckerProcess::processHttpCheckResult( Result<CheckStatusInfo> result = None(); if (future.isReady()) { - VLOG(1) << name << " for task '" << taskId << "'" - << " returned: " << future.get(); + LOG(INFO) << name << " for task '" << taskId << "'" + << " returned: " << future.get(); CheckStatusInfo checkStatusInfo; checkStatusInfo.set_type(check.type()); @@ -1099,8 +1099,8 @@ void CheckerProcess::processTcpCheckResult( Result<CheckStatusInfo> result = None(); if (future.isReady()) { - VLOG(1) << name << " for task '" << taskId << "'" - << " returned: " << future.get(); + LOG(INFO) << name << " for task '" << taskId << "'" + << " returned: " << future.get(); CheckStatusInfo checkStatusInfo; checkStatusInfo.set_type(check.type());
