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

List:       openjdk-serviceability-dev
Subject:    RE: JDK-8217473: SA: Tests using ClhsdbLauncher fail on SAP docker containers
From:       "Lindenmaier, Goetz" <goetz.lindenmaier () sap ! com>
Date:       2019-01-31 15:11:20
Message-ID: bd4acbfed98342ad82539c611573be96 () sap ! com
[Download RAW message or body]

... and the tests passed.
https://ci.sapmachine.io/job/branch-build-linux_x86_64/12/JT_20Report_20hotspot/

Unfortunately one can not see the jtr file to verify the right exception was 
thrown.  But they passed, so it's good. 

Why do you remove some checks for test failures from some tests?
Like 
  serviceability/sa/ClhsdbInspect.java
Why won't jstackOutput be null any more after your change?

Others are:
  serviceability/sa/ClhsdbJdis.java
  serviceability/sa/ClhsdbLongConstant.java
  serviceability/sa/ClhsdbPrintAs.java
  serviceability/sa/ClhsdbRegionDetailsScanOopsForG1.java
  serviceability/sa/ClhsdbScanOops.java
  serviceability/sa/ClhsdbThread.java

Are these all failures that go back to the attach issue?

Best regards,
  Goetz.


> -----Original Message-----
> From: Lindenmaier, Goetz
> Sent: Donnerstag, 31. Januar 2019 12:54
> To: 'Jini George' <jini.george@oracle.com>; serviceability-
> dev@openjdk.java.net
> Subject: RE: JDK-8217473: SA: Tests using ClhsdbLauncher fail on SAP docker
> containers
> 
> Hi Jini,
> 
> I pushed it into our testing:
> https://ci.sapmachine.io/view/Build%20My%20Branch/job/branch-build-
> service/43/
> 
> the build is running ...
> 
> Best regards,
>   Goetz.
> 
> > -----Original Message-----
> > From: serviceability-dev <serviceability-dev-bounces@openjdk.java.net> On
> > Behalf Of Jini George
> > Sent: Mittwoch, 30. Januar 2019 12:59
> > To: serviceability-dev@openjdk.java.net
> > Subject: RFR: JDK-8217473: SA: Tests using ClhsdbLauncher fail on SAP
> docker
> > containers
> >
> > Requesting reviews for:
> >
> > BugID: https://bugs.openjdk.java.net/browse/JDK-8217473
> > Webrev:http://cr.openjdk.java.net/~jgeorge/8217473/webrev.01/index.html
> >
> > The patch includes changes to use SkippedException to skip the tests
> > when canPtraceAttachLinux() in Platform.java returns false.
> >
> > Thanks,
> > Jini.


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