[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Re: RFR: JDK-8047973 Quarantine compiler/ciReplay/*
From: Vladimir Ivanov <vladimir.x.ivanov () oracle ! com>
Date: 2014-06-30 7:49:52
Message-ID: 53B116A0.4030301 () oracle ! com
[Download RAW message or body]
Reviewed.
Best regards,
Vladimir Ivanov
On 6/30/14 11:42 AM, Staffan Larsen wrote:
> Can I have a Review of this change?
>
> Thanks,
> /Staffan
>
>
> On 25 jun 2014, at 08:53, Staffan Larsen <staffan.larsen@oracle.com> wrote:
>
> >
> > On 24 jun 2014, at 14:45, Igor Ignatyev <igor.ignatyev@oracle.com> wrote:
> >
> > > Staffan,
> > >
> > > 1. '@ignore' should be followed by CR numbers
> >
> > Good catch - I have fixed that below.
> >
> > > 2. I doubt that TestVM* failures are related to JDK-8032226. these tests don't \
> > > use SA agent, they rather fail due to JDK-8031978
> >
> > You are right. There seems to be some confusion in the RULEs for these two CRs so \
> > they match incorrectly. I have added 8031978 or 8032226 as appropriate to the \
> > @ignores.
> > /Staffan
> >
> >
> > diff --git a/test/compiler/ciReplay/TestSA.sh b/test/compiler/ciReplay/TestSA.sh
> > --- a/test/compiler/ciReplay/TestSA.sh
> > +++ b/test/compiler/ciReplay/TestSA.sh
> > @@ -26,6 +26,7 @@
> > ##
> > ## @test
> > ## @bug 8011675
> > +## @ignore 8032226, 8031978
> > ## @summary testing of ciReplay with using generated by SA replay.txt
> > ## @author igor.ignatyev@oracle.com
> > ## @run shell TestSA.sh
> > diff --git a/test/compiler/ciReplay/TestVM.sh b/test/compiler/ciReplay/TestVM.sh
> > --- a/test/compiler/ciReplay/TestVM.sh
> > +++ b/test/compiler/ciReplay/TestVM.sh
> > @@ -26,6 +26,7 @@
> > ##
> > ## @test
> > ## @bug 8011675
> > +## @ignore 8031978
> > ## @summary testing of ciReplay with using generated by VM replay.txt
> > ## @author igor.ignatyev@oracle.com
> > ## @run shell TestVM.sh
> > diff --git a/test/compiler/ciReplay/TestVM_no_comp_level.sh \
> > b/test/compiler/ciReplay/TestVM_no_comp_level.sh
> > --- a/test/compiler/ciReplay/TestVM_no_comp_level.sh
> > +++ b/test/compiler/ciReplay/TestVM_no_comp_level.sh
> > @@ -26,6 +26,7 @@
> > ##
> > ## @test
> > ## @bug 8011675
> > +## @ignore 8031978
> > ## @summary testing of ciReplay with using generated by VM replay.txt w/o \
> > comp_level ## @author igor.ignatyev@oracle.com
> > ## @run shell TestVM_no_comp_level.sh
> >
> > >
> > > Igor
> > >
> > > On 06/24/2014 03:12 PM, Staffan Larsen wrote:
> > > > These tests keep being unstable in nightly testing (see JDK-8032226) so will \
> > > > have to be quarantined.
> > > > Thanks,
> > > > /Staffan
> > > >
> > > >
> > > >
> > > > diff --git a/test/compiler/ciReplay/TestSA.sh \
> > > > b/test/compiler/ciReplay/TestSA.sh
> > > > --- a/test/compiler/ciReplay/TestSA.sh
> > > > +++ b/test/compiler/ciReplay/TestSA.sh
> > > > @@ -26,6 +26,7 @@
> > > > ##
> > > > ## @test
> > > > ## @bug 8011675
> > > > +## @ignore
> > > > ## @summary testing of ciReplay with using generated by SA replay.txt
> > > > ## @author igor.ignatyev@oracle.com
> > > > ## @run shell TestSA.sh
> > > > diff --git a/test/compiler/ciReplay/TestVM.sh \
> > > > b/test/compiler/ciReplay/TestVM.sh
> > > > --- a/test/compiler/ciReplay/TestVM.sh
> > > > +++ b/test/compiler/ciReplay/TestVM.sh
> > > > @@ -26,6 +26,7 @@
> > > > ##
> > > > ## @test
> > > > ## @bug 8011675
> > > > +## @ignore
> > > > ## @summary testing of ciReplay with using generated by VM replay.txt
> > > > ## @author igor.ignatyev@oracle.com
> > > > ## @run shell TestVM.sh
> > > > diff --git a/test/compiler/ciReplay/TestVM_no_comp_level.sh \
> > > > b/test/compiler/ciReplay/TestVM_no_comp_level.sh
> > > > --- a/test/compiler/ciReplay/TestVM_no_comp_level.sh
> > > > +++ b/test/compiler/ciReplay/TestVM_no_comp_level.sh
> > > > @@ -26,6 +26,7 @@
> > > > ##
> > > > ## @test
> > > > ## @bug 8011675
> > > > +## @ignore
> > > > ## @summary testing of ciReplay with using generated by VM replay.txt w/o \
> > > > comp_level ## @author igor.ignatyev@oracle.com
> > > > ## @run shell TestVM_no_comp_level.sh
> > > >
> >
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic