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;