Thanks for spotting it. New webrev at http://cr.openjdk.java.net/~miauno/8040748/webrev.03/.
Mikael On 2014-05-05 16:18, Staffan Larsen wrote: > 8039432 is Resolved and should not be listed. > > /Staffan > > On 5 maj 2014, at 15:26, Mikael Auno <[email protected]> wrote: > >> I've updated this change to use ProblemList.txt instead of @ignore >> (since that was only intended for hotspot/test and not jdk/test). I've >> also updated the data to nightly failures from the last two weeks with >> issues since then filtered out manually. >> >> Hopefully this should address all concerns. >> >> New webrev at http://cr.openjdk.java.net/~miauno/8040748/webrev.02/ >> >> Thanks, >> Mikael >> >> On 2014-04-25 08:15, Staffan Larsen wrote: >>> For test/com/sun/jdi/JdbReadTwiceTest.sh, you list JDK-8002116, but that >>> bug is resolved so should not be reason to quarantine the test. >>> >>> For test/com/sun/jdi/RepStep.java you list JDK-6471769 JDK-6766320 but both >>> of those are closed so should not be reason to quarantine the test. >>> >>> For test/demo/jvmti/mtrace/TraceJFrame.java you list JDK-8035195 and >>> JDK-8039432, but 8035195 seems to be just an old name for 8039432 (looking >>> it up in JBS takes you to 8039432). So 8035195 should not be listed. >>> >>> For test/sun/tools/jinfo/Basic.sh, there are no open bugs against it >>> (except JDK-6542634 which is a timeout from 2007 and I’m not sure it is >>> valid) so I don’t think it should be quarantined. >>> >>> /Staffan >>> >>> >>> On 24 apr 2014, at 20:01, Mikael Auno <[email protected]> wrote: >>> >>>> I've now added the keyword "quarantine" as well as fixed an issue with >>>> placement of the @ignore tag (it's not allowed to come before @library). >>>> >>>> New webrev at http://cr.openjdk.java.net/~miauno/8040748/webrev.01/ >>>> >>>> Local testing: >>>> >>>>> % jtreg -jdk $JAVA_HOME -k:quarantine -ignore:quiet -verbose:summary >>>>> /localhome/ws/jdk9-dev/jdk/test/ >>>>> Test results: no tests selected >>>>> Report written to /localhome/temp/run/JTreport/html/report.html >>>>> Results written to /localhome/temp/run/JTwork >>>> >>>> Thanks, >>>> Mikael >>>> >>>> On 2014-04-22 17:12, Stefan Särne wrote: >>>>> Hi Mikael, >>>>> >>>>> I think we should use a key word to group quarantined tests, which can be >>>>> used to run the tests as a separate batch. Recommend to add this to all >>>>> tests: >>>>> @key quarantine >>>>> >>>>> Note that you may have to add the key word as a known key word to the >>>>> TEST.ROOT file as well. >>>>> >>>>> Best regards, >>>>> Stefan >>>>> >>>>> -----Original Message----- >>>>> From: Mikael Auno >>>>> Sent: den 16 april 2014 19:51 >>>>> To: [email protected] >>>>> Subject: RFR 8040748: [TESTBUG] Exclude failing (serviceability) jtreg >>>>> tests using @ignore >>>>> >>>>> Please, review the following fix adding the @ignore tag to a couple of >>>>> serviceability tests >>>>> >>>>> Issue: https://bugs.openjdk.java.net/browse/JDK-8040748 >>>>> Webrev: http://cr.openjdk.java.net/~miauno/8040748/webrev.00/ >>>>> >>>>> Thanks, >>>>> Mikael >>>>> >>>> >>> >> >
