You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ji...@apache.org on 2016/10/01 00:14:52 UTC
[2/4] mesos git commit: Ignored cleanup request for unknown
containers in posix isolators.
Ignored cleanup request for unknown containers in posix isolators.
To be consistent with other isolators. If the operator changes
isolation flags, the cleanup can be called for a container that is
unknown to the new isolator.
Review: https://reviews.apache.org/r/52460
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/3dca7be3
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/3dca7be3
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/3dca7be3
Branch: refs/heads/master
Commit: 3dca7be322e3d27f008681b148e3044e37201389
Parents: 012d6d8
Author: Jie Yu <yu...@gmail.com>
Authored: Fri Sep 30 15:51:23 2016 -0700
Committer: Jie Yu <yu...@gmail.com>
Committed: Fri Sep 30 16:17:38 2016 -0700
----------------------------------------------------------------------
src/slave/containerizer/mesos/isolators/posix.hpp | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/3dca7be3/src/slave/containerizer/mesos/isolators/posix.hpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/isolators/posix.hpp b/src/slave/containerizer/mesos/isolators/posix.hpp
index 26c693a..6270046 100644
--- a/src/slave/containerizer/mesos/isolators/posix.hpp
+++ b/src/slave/containerizer/mesos/isolators/posix.hpp
@@ -116,7 +116,10 @@ public:
virtual process::Future<Nothing> cleanup(const ContainerID& containerId)
{
if (!promises.contains(containerId)) {
- return process::Failure("Unknown container: " + stringify(containerId));
+ VLOG(1) << "Ignoring cleanup request for unknown container "
+ << containerId;
+
+ return Nothing();
}
// TODO(idownes): We should discard the container's promise here to signal