You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by al...@apache.org on 2016/08/13 01:46:10 UTC
[6/6] mesos git commit: Removed extra `HealthCheck` validation in
docker executor.
Removed extra `HealthCheck` validation in docker executor.
The extra validation of the `HealthCheck` protobuf in the docker
executor is not necessary. The `HealthCheck` protobuf will be first
validated by the master as part of the `TaskInfo` validation and then
the modified version of the protobuf by the health check library.
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/a536cef6
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/a536cef6
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/a536cef6
Branch: refs/heads/master
Commit: a536cef67d2f250f60eb2f991e62402dae0590a1
Parents: 9536178
Author: Alexander Rukletsov <al...@apache.org>
Authored: Sat Aug 13 03:41:45 2016 +0200
Committer: Alexander Rukletsov <al...@apache.org>
Committed: Sat Aug 13 03:43:19 2016 +0200
----------------------------------------------------------------------
src/docker/executor.cpp | 3 ---
1 file changed, 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/a536cef6/src/docker/executor.cpp
----------------------------------------------------------------------
diff --git a/src/docker/executor.cpp b/src/docker/executor.cpp
index ba70749..4a755f3 100644
--- a/src/docker/executor.cpp
+++ b/src/docker/executor.cpp
@@ -481,9 +481,6 @@ private:
HealthCheck healthCheck = task.health_check();
- // Validate the `HealthCheck` protobuf.
- Option<Error> validateCheck = health::validation::healthCheck(healthCheck);
-
if (validateCheck.isSome()) {
cerr << "Unable to launch health check process: "
<< validateCheck.get().message << endl;