[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