[qpid-proton] branch master updated: PROTON-2115 Set only the soft process limit in fdlimit test

[jd...@apache.org]

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

jdanek pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/qpid-proton.git


The following commit(s) were added to refs/heads/master by this push:
     new fee97b1  PROTON-2115 Set only the soft process limit in fdlimit test
fee97b1 is described below

commit fee97b14c9af0ac0e8dabdc48c7f86f5190deb57
Author: Jiri Danek <jd...@redhat.com>
AuthorDate: Thu Nov 7 15:28:48 2019 +0100

    PROTON-2115 Set only the soft process limit in fdlimit test
    
    Otherwise, when setting both the soft and hard limit it may fail with
    
        prlimit: failed to set the NOFILE resource limit: Operation not permitted
---
 c/tests/fdlimit.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/c/tests/fdlimit.py b/c/tests/fdlimit.py
index c4408b9..349ab0b 100644
--- a/c/tests/fdlimit.py
+++ b/c/tests/fdlimit.py
@@ -38,7 +38,7 @@ except OSError:
 class PRLimitedBroker(test_subprocess.Server):
     def __init__(self, fdlimit, *args, **kwargs):
         super(PRLimitedBroker, self).__init__(
-            ['prlimit', '-n{0:d}'.format(fdlimit), "broker", "", "0"],
+            ['prlimit', '-n{0:d}:'.format(fdlimit), "broker", "", "0"],  # `-n 256:` sets only soft limit to 256
             stdout=subprocess.PIPE, universal_newlines=True, *args, **kwargs)
         self.fdlimit = fdlimit
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org