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/89747f4a Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/89747f4a Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/89747f4a Branch: refs/heads/1.2.x Commit: 89747f4a0ae6bfdb96d7d7db70b7537755c5d4bc Parents: 959125a 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 01:29:48 2017 +0200 ---------------------------------------------------------------------- src/docker/executor.cpp | 5 +++++ 1 file changed, 5 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/89747f4a/src/docker/executor.cpp ---------------------------------------------------------------------- diff --git a/src/docker/executor.cpp b/src/docker/executor.cpp index 6786687..e638451 100644 --- a/src/docker/executor.cpp +++ b/src/docker/executor.cpp @@ -374,6 +374,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; + })); } }
