[mesos] branch master updated: Fixed the ulimit validation in the parallel test runner.

[ch...@apache.org]

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

chhsiao 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 9df27e3  Fixed the ulimit validation in the parallel test runner.
9df27e3 is described below

commit 9df27e3657aa44677668baaee70e46b7963b06e4
Author: Chun-Hung Hsiao <ch...@apache.org>
AuthorDate: Fri Nov 9 11:47:01 2018 -0800

    Fixed the ulimit validation in the parallel test runner.
    
    This patch fixes the following error:
    ```
    $ support/mesos-gtest-runner.py build/bin/mesos-tests.sh
    Could not check compatibility of ulimit settings: \
    [Errno 2] No such file or directory: 'ulimit': 'ulimit'
    ```
    
    Review: https://reviews.apache.org/r/69301/
---
 support/mesos-gtest-runner.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/support/mesos-gtest-runner.py b/support/mesos-gtest-runner.py
index d2a57ee..135c488 100755
--- a/support/mesos-gtest-runner.py
+++ b/support/mesos-gtest-runner.py
@@ -206,7 +206,7 @@ def validate_setup(options):
 
     # Check the number of processes/threads a user is allowed to execute.
     try:
-        ulimit = int(subprocess.check_output(['ulimit', '-u']))
+        ulimit = int(subprocess.check_output('ulimit -u', shell=True))
 
         # As a proxy for our requirements we assume that each test executable
         # launches a thread for every available CPU times a factor of 16 to