Repository: mesos Updated Branches: refs/heads/1.3.x f38bfbf34 -> f2deb04a5
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/5852756b Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/5852756b Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/5852756b Branch: refs/heads/1.3.x Commit: 5852756ba6faa9f9704c677901a773f4c98109d8 Parents: f38bfbf Author: Andrei Budnik <abud...@mesosphere.com> Authored: Thu Aug 10 18:52:51 2017 +0200 Committer: Alexander Rukletsov <al...@apache.org> Committed: Fri Aug 11 00:08:53 2017 +0200 ---------------------------------------------------------------------- src/docker/executor.cpp | 5 +++++ 1 file changed, 5 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/5852756b/src/docker/executor.cpp ---------------------------------------------------------------------- diff --git a/src/docker/executor.cpp b/src/docker/executor.cpp index 82ae9bd..875b43c 100644 --- a/src/docker/executor.cpp +++ b/src/docker/executor.cpp @@ -406,6 +406,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; + })); } }
