You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2019/10/24 13:50:00 UTC

[jira] [Commented] (PROTON-2115) Bitrot in qpid-proton/c/tests/fdlimit.py

    [ https://issues.apache.org/jira/browse/PROTON-2115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16958905#comment-16958905 ] 

ASF GitHub Bot commented on PROTON-2115:
----------------------------------------

jdanekrh commented on pull request #198: PROTON-2115 [c] fix prlimit test and do PEP8 reformat in related files
URL: https://github.com/apache/qpid-proton/pull/198
 
 
   This fixes prlimit test, adds some more useful 2.7 unittest methods, and does some PEP8 formatting around the files.
   
   cc @rkubis 
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Bitrot in qpid-proton/c/tests/fdlimit.py
> ----------------------------------------
>
>                 Key: PROTON-2115
>                 URL: https://issues.apache.org/jira/browse/PROTON-2115
>             Project: Qpid Proton
>          Issue Type: Test
>    Affects Versions: proton-c-0.29.0
>            Reporter: Jiri Daněk
>            Assignee: Jiri Daněk
>            Priority: Major
>
> Originally, {{Proc}} was imported in scope, https://github.com/apache/qpid-proton/commit/087b94faef6f0bc3fc2eac50ba0d2c2b03d82fbd#diff-9ad88150e74c307cdd1f621348dca9d6R20
> Nowadays, it isn't, current state of the file is this https://github.com/apache/qpid-proton/blob/9778eda8e9387caa8a207fbb6fedfa09ac0e298a/c/tests/fdlimit.py#L41 
> {noformat}
> # Check if we can run prlimit to control resources
> try:
>     Proc(["prlimit"]).wait_exit()
> except:
>     print("Skipping test: prlimit not available")
>     sys.exit(0)
> {noformat}
> The broad except masks the error here, and also anything else in the rest of the file (which is never executed).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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