[jira] Updated: (ZOOKEEPER-820) update c unit tests to ensure "zombie" java server processes don't cause failure

["Michi Mutsuzaki (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ZOOKEEPER-820?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Michi Mutsuzaki updated ZOOKEEPER-820:
--------------------------------------

    Attachment: ZOOKEEPER-820.patch

This patch is for branch-3.3.

I changed 2 things.

1. Instead of using a file to keep track of pid, use lsof to check if a process is using port 22181.
2. Fixed invalid return code checking. This was causing the unit test to go into a loop for 2 minutes in certain cases. 

--Michi

> update c unit tests to ensure "zombie" java server processes don't cause failure
> --------------------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-820
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-820
>             Project: Zookeeper
>          Issue Type: Bug
>    Affects Versions: 3.3.1
>            Reporter: Patrick Hunt
>            Assignee: Michi Mutsuzaki
>            Priority: Critical
>             Fix For: 3.3.2, 3.4.0
>
>         Attachments: ZOOKEEPER-820.patch
>
>
> When the c unit tests are run sometimes the server doesn't shutdown at the end of the test, this causes subsequent tests (hudson esp) to fail.
> 1) we should try harder to make the server shut down at the end of the test, I suspect this is related to test failing/cleanup
> 2) before the tests are run we should see if the old server is still running and try to shut it down

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.