Repository: mesos
Updated Branches:
  refs/heads/1.1.x 33a418128 -> 8e79521d6


Added logging in docker executor on `docker stop` failure.

Review: https://reviews.apache.org/r/61435/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/c43cfbb9
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/c43cfbb9
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/c43cfbb9

Branch: refs/heads/1.1.x
Commit: c43cfbb95d637fa6d9264014077794ab8a2a884f
Parents: 33a4181
Author: Andrei Budnik <abud...@mesosphere.com>
Authored: Thu Aug 10 18:52:51 2017 +0200
Committer: Alexander Rukletsov <al...@apache.org>
Committed: Thu Aug 10 22:50:06 2017 +0200

----------------------------------------------------------------------
 src/docker/executor.cpp | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/c43cfbb9/src/docker/executor.cpp
----------------------------------------------------------------------
diff --git a/src/docker/executor.cpp b/src/docker/executor.cpp
index 3eea3e8..cf2523c 100644
--- a/src/docker/executor.cpp
+++ b/src/docker/executor.cpp
@@ -380,6 +380,11 @@ private:
       // that we can adjust the grace period in the case of
       // a `KillPolicy` override.
       stop = docker->stop(containerName, gracePeriod);
+
+      stop.onFailed(defer(self(), [=](const string& failure) {
+        LOG(ERROR) << "Failed to stop container '" << containerName << "'"
+                   << ": " << failure;
+      }));
     }
   }
 

Reply via email to