Can I have a Review of this change? Thanks, /Staffan
On 25 jun 2014, at 08:53, Staffan Larsen <staffan.lar...@oracle.com> wrote: > > On 24 jun 2014, at 14:45, Igor Ignatyev <igor.ignat...@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.ignat...@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.ignat...@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.ignat...@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.ignat...@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.ignat...@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.ignat...@oracle.com >>> ## @run shell TestVM_no_comp_level.sh >>> >