This is an automated email from the ASF dual-hosted git repository.

bbannier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/master by this push:
     new 21ccad2  Disabled `DefaultExecutorTest.DomainSockets` on non-Linux 
platforms.
21ccad2 is described below

commit 21ccad220f04369a7accf2bafae8f1d5002646bb
Author: Benjamin Bannier <[email protected]>
AuthorDate: Mon Jan 13 11:18:49 2020 +0100

    Disabled `DefaultExecutorTest.DomainSockets` on non-Linux platforms.
    
    While the interfaces are available on all platforms, currently domain
    sockets for executor communication are only available on Linux. This
    patch ifdef's the code away on non-Linux platforms; this is consistent
    with how we disable tests on non-Linux platforms elsewhere where we
    never use a test filter.
    
    Review: https://reviews.apache.org/r/71987
---
 src/tests/default_executor_tests.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/tests/default_executor_tests.cpp 
b/src/tests/default_executor_tests.cpp
index 6c71b3c..2353520 100644
--- a/src/tests/default_executor_tests.cpp
+++ b/src/tests/default_executor_tests.cpp
@@ -4418,6 +4418,7 @@ TEST_P(DefaultExecutorTest, 
AllocationRoleEnvironmentVariable)
 }
 
 
+#ifdef __linux__
 // This test verifies that the default executor will attempt to connect to
 // the domain socket located at `MESOS_DOMAIN_SOCKET` if that environment
 // variable is set.
@@ -4490,6 +4491,7 @@ TEST_P(DefaultExecutorTest, DomainSockets)
   Try<Nothing> rmdir = os::rmdir(socketDir.get());
   ASSERT_SOME(rmdir);
 }
+#endif
 
 } // namespace tests {
 } // namespace internal {

Reply via email to