[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: JDK-8194067 - [Testbug] serviceability/sa/Jhsdb* tests can't tolerate unrelated warnings
From:       "serguei.spitsyn () oracle ! com" <serguei ! spitsyn () oracle ! com>
Date:       2018-01-09 17:59:31
Message-ID: 15c74984-fdc2-bc0c-66ec-3fb7c0e04ac6 () oracle ! com
[Download RAW message or body]

+1

Thanks,
Serguei


On 1/9/18 01:18, David Holmes wrote:
> Looks good!
>
> Thanks,
> David
>
> On 9/01/2018 6:39 PM, Sharath Ballal wrote:
>> Thanks for the comments David.   I have made the changes.
>>
>> Webrev: http://cr.openjdk.java.net/~sballal/8194067/webrev.02/
>>
>>
>> Thanks,
>> Sharath
>>
>>
>> -----Original Message-----
>> From: David Holmes
>> Sent: Monday, January 08, 2018 5:11 PM
>> To: Sharath Ballal; serviceability-dev@openjdk.java.net
>> Subject: Re: RFR: JDK-8194067 - [Testbug] serviceability/sa/Jhsdb* 
>> tests can't tolerate unrelated warnings
>>
>> Hi Sharath,
>>
>> On 8/01/2018 3:30 PM, Sharath Ballal wrote:
>>> Thanks David.   I have made the changes.
>>> New webrev is http://cr.openjdk.java.net/~sballal/8194067/webrev.01/
>>
>> That looks quite good - only issue is that the regex won't work for 
>> an OpenJDK build as it produces:
>>
>> OpenJDK 64-Bit Server VM warning: ...
>>
>> rather than
>>
>> Java HotSpot(TM) 64-Bit Server VM warning: ...
>>
>> Maybe "VM warning:" will suffice for the match?
>>
>> Thanks,
>> David
>> -----
>>
>>> Thanks,
>>> Sharath
>>>
>>>
>>> -----Original Message-----
>>> From: David Holmes
>>> Sent: Friday, December 29, 2017 6:13 AM
>>> To: Sharath Ballal; serviceability-dev@openjdk.java.net
>>> Subject: Re: RFR: JDK-8194067 - [Testbug] serviceability/sa/Jhsdb*
>>> tests can't tolerate unrelated warnings
>>>
>>> When I reported this one I thought about how to best fix it. What 
>>> you propose is a simple fix, but may potentially allow other errors 
>>> to go unnoticed. I was wondering whether there was a simple way to 
>>> filter the stderr content to exclude VM Warnings, but still check 
>>> for other unexpected content?
>>>
>>> Thanks,
>>> David
>>>
>>> On 28/12/2017 3:29 PM, Sharath Ballal wrote:
>>>> Hello,
>>>>
>>>> Requesting reviews for:
>>>>
>>>> JBS Id: https://bugs.openjdk.java.net/browse/JDK-8194067
>>>>
>>>> Webrev: http://cr.openjdk.java.net/~sballal/8194067/webrev.00/
>>>>
>>>> The tests ran successfully in JPRT and Mach5.
>>>>
>>>> Thanks,
>>>>
>>>> Sharath
>>>>

[prev in list] [next in list] [prev in thread] [next in thread]