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/07 16:38:59 UTC

[09/10] mesos git commit: Removed some redundant cleanup code in linux filesystem isolator.

Removed some redundant cleanup code in linux filesystem isolator.

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


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

Branch: refs/heads/master
Commit: a3fd145960069d837e8b79f557b11bfac59ab98b
Parents: f3c574d
Author: Jie Yu <yu...@gmail.com>
Authored: Tue Oct 4 23:26:20 2016 -0700
Committer: Jie Yu <yu...@gmail.com>
Committed: Fri Oct 7 09:31:17 2016 -0700

----------------------------------------------------------------------
 .../containerizer/linux_filesystem_isolator_tests.cpp | 14 --------------
 1 file changed, 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/a3fd1459/src/tests/containerizer/linux_filesystem_isolator_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/containerizer/linux_filesystem_isolator_tests.cpp b/src/tests/containerizer/linux_filesystem_isolator_tests.cpp
index bf8a44d..b522037 100644
--- a/src/tests/containerizer/linux_filesystem_isolator_tests.cpp
+++ b/src/tests/containerizer/linux_filesystem_isolator_tests.cpp
@@ -71,20 +71,6 @@ namespace tests {
 class LinuxFilesystemIsolatorTest : public MesosTest
 {
 protected:
-  virtual void TearDown()
-  {
-    // Try to remove any mounts under sandbox.
-    if (::geteuid() == 0) {
-      Try<Nothing> unmount = fs::unmountAll(sandbox.get(), MNT_DETACH);
-      if (unmount.isError()) {
-        LOG(ERROR) << "Failed to umount for sandbox '" << sandbox.get()
-                   << "': " << unmount.error();
-      }
-    }
-
-    MesosTest::TearDown();
-  }
-
   Fetcher fetcher;
 };