RFR 9: 8130296 [TESTBUG] java/lang/ProcessHandle/OnExitTest - Unaccounted for children

[Joseph D. Darcy]

Looks good Roger; thanks,

-Joe

On 7/8/2015 2:08 PM, Roger Riggs wrote:
> Hi Joe,
>
> Webrev updated in place.
>
> The jdk.testlibrary.Utils.adjustTimeout(timeout) returns the timeout 
> adjusted
> by the jtreg -timeoutFactor argument.
>
> Thanks, Roger
>
>
> On 7/8/2015 4:50 PM, joe darcy wrote:
>> Hi Roger,
>>
>> I think the test would be more robust if the hard-coded time out 
>> window was a function of the time factor the test was running under. 
>> Maybe
>>
>>     10 seconds * timeout
>>
>> IIRC, the timeout factor is available as an environment variable (or 
>> similar) in the running tests. I checked the jtreg docs quickly and 
>> didn't find it, but I believe the jdk regression tests have already 
>> had a pass to do this kind of update so there should be some guiding 
>> examples in the sources.
>>
>> Thanks,
>>
>> -Joe
>>
>> On 7/8/2015 1:16 PM, Roger Riggs wrote:
>>> Ping and note this change is simplified after the fix for 8085981.
>>>
>>> Please review this change to a ProcessHandle test to address 
>>> intermittent failures.
>>> Unexpected subprocesses has been observed. The test succeeds if all 
>>> of the
>>> processes under test are accounted for; others are ignored.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~rriggs/webrev-exit-8130296/
>>>
>>> Issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8130296
>>>
>>> Thanks, Roger
>>>
>>>
>>
>