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 2017/12/09 02:54:50 UTC

[3/7] mesos git commit: Added missing filesystem isolation for SLRP tests.

Added missing filesystem isolation for SLRP tests.

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


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

Branch: refs/heads/master
Commit: 4c5c7ce03e854818814a7327f62231825337190e
Parents: 6f33a1f
Author: Chun-Hung Hsiao <ch...@mesosphere.io>
Authored: Fri Dec 8 18:12:17 2017 -0800
Committer: Jie Yu <yu...@gmail.com>
Committed: Fri Dec 8 18:54:43 2017 -0800

----------------------------------------------------------------------
 src/Makefile.am                                     | 6 +++++-
 src/tests/storage_local_resource_provider_tests.cpp | 4 +++-
 2 files changed, 8 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/4c5c7ce0/src/Makefile.am
----------------------------------------------------------------------
diff --git a/src/Makefile.am b/src/Makefile.am
index 1620b2a..47f4528 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -2617,9 +2617,13 @@ if ENABLE_GRPC
 mesos_tests_SOURCES +=						\
   tests/csi_client_tests.cpp					\
   tests/mock_csi_plugin.cpp					\
-  tests/mock_csi_plugin.hpp					\
+  tests/mock_csi_plugin.hpp
+
+if OS_LINUX
+mesos_tests_SOURCES +=						\
   tests/storage_local_resource_provider_tests.cpp
 endif
+endif
 
 if ENABLE_SSL
 mesos_tests_SOURCES +=						\

http://git-wip-us.apache.org/repos/asf/mesos/blob/4c5c7ce0/src/tests/storage_local_resource_provider_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/storage_local_resource_provider_tests.cpp b/src/tests/storage_local_resource_provider_tests.cpp
index 0f0c627..e5868bf 100644
--- a/src/tests/storage_local_resource_provider_tests.cpp
+++ b/src/tests/storage_local_resource_provider_tests.cpp
@@ -64,7 +64,7 @@ public:
           "name": "test",
           "default_reservations": [
             {
-              "type": "STATIC",
+              "type": "DYNAMIC",
               "role": "storage"
             }
           ],
@@ -120,6 +120,7 @@ TEST_F(StorageLocalResourceProviderTest, ROOT_CreateVolumeAndDestroyVolume)
   Owned<MasterDetector> detector = master.get()->createDetector();
 
   slave::Flags flags = CreateSlaveFlags();
+  flags.isolation = "filesystem/linux";
 
   // Disable HTTP authentication to simplify resource provider interactions.
   flags.authenticate_http_readwrite = false;
@@ -297,6 +298,7 @@ TEST_F(StorageLocalResourceProviderTest, ROOT_LaunchAndDestroyVolume)
   Owned<MasterDetector> detector = master.get()->createDetector();
 
   slave::Flags flags = CreateSlaveFlags();
+  flags.isolation = "filesystem/linux";
 
   // Disable HTTP authentication to simplify resource provider interactions.
   flags.authenticate_http_readwrite = false;