[11u] RFR: 8227642: [TESTBUG] Make docker tests podman compatible

[Severin Gehwolf]

Hi Christoph,

On Tue, 2019-09-24 at 14:37 +0000, Langer, Christoph wrote:
> Hi Severin,
> 
> thanks for review and this hint. I've just created a backport patch
> and will run it through our nightlies. Will probably send out an RFR
> tomorrow.
> 
> Interestingly we didn't encounter issues in jdk/net/Sockets/Test.java
> with 8227642 applied. But in any case, adding 8228434 will be better.

On second thought, it looks like the OpenJDK 11u version of
jdk/net/Sockets/Test.java doesn't use Platform (jdk/jdk's version
does). So we are probably good at this point. No need for 8228434 in
JDK 11u, AFAICS. Usages of Platform + running with a SecurityManager
triggers the problem. OpenJDK 11 seems OK.

Thanks,
Severin

> 
> > -----Original Message-----
> > From: Severin Gehwolf <sgehwolf at redhat.com>
> > Sent: Dienstag, 24. September 2019 16:12
> > To: Langer, Christoph <christoph.langer at sap.com>; jdk-updates-
> > dev at openjdk.java.net
> > Cc: hotspot-dev at openjdk.java.net
> > Subject: Re: [11u] RFR: 8227642: [TESTBUG] Make docker tests podman
> > compatible
> > 
> > Hi Christoph,
> > 
> > On Tue, 2019-09-24 at 13:59 +0000, Langer, Christoph wrote:
> > > Hi,
> > > 
> > > please help reviewing the backport of test fix 8227642: [TESTBUG]
> > > Make docker tests podman compatible.
> > > 
> > > Bug: https://bugs.openjdk.java.net/browse/JDK-8227642
> > > Original Change: https://hg.openjdk.java.net/jdk/jdk/rev/709913d8ace9
> > > Webrev:
> > > http://cr.openjdk.java.net/~clanger/webrevs/8227642.11u-dev.0/
> > 
> > Looks good.
> > 
> > Please also backport JDK-8228434[1] as otherwise we risk breaking the
> > jdk/net/Sockets/Test.java test.
> > 
> > Thanks,
> > Severin
> > 
> > [1] https://bugs.openjdk.java.net/browse/JDK-8228434