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 2018/03/27 09:38:47 UTC
[5/8] mesos git commit: Made the default executor log kill failures.
Made the default executor log kill failures.
Review: https://reviews.apache.org/r/66233/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/7d0b4489
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/7d0b4489
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/7d0b4489
Branch: refs/heads/master
Commit: 7d0b4489bb940bf0b116342589b22edc247b65b9
Parents: 501eb34
Author: Gaston Kleiman <ga...@mesosphere.io>
Authored: Tue Mar 27 11:37:44 2018 +0200
Committer: Alexander Rukletsov <al...@apache.org>
Committed: Tue Mar 27 11:37:44 2018 +0200
----------------------------------------------------------------------
src/launcher/default_executor.cpp | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/7d0b4489/src/launcher/default_executor.cpp
----------------------------------------------------------------------
diff --git a/src/launcher/default_executor.cpp b/src/launcher/default_executor.cpp
index a2f1ec5..57bcdf9 100644
--- a/src/launcher/default_executor.cpp
+++ b/src/launcher/default_executor.cpp
@@ -1245,7 +1245,13 @@ protected:
return;
}
- kill(container.get(), killPolicy);
+ const ContainerID& containerId = container->containerId;
+ kill(container.get(), killPolicy)
+ .onFailed(defer(self(), [=](const string& failure) {
+ LOG(WARNING) << "Failed to kill the task '" << taskId
+ << "' running in child container " << containerId << ": "
+ << failure;
+ }));
}
void taskCheckUpdated(