[PING-2] RFR 6425769: jmx remote bind address

[Severin Gehwolf]

On Wed, 2015-12-09 at 14:58 +0100, Jaroslav Bachorik wrote:
> On 9.12.2015 14:55, Severin Gehwolf wrote:
> > On Tue, 2015-12-01 at 14:19 +0100, Severin Gehwolf wrote:
> > > Hi Jaroslav,
> > > 
> > > On Tue, 2015-12-01 at 12:33 +0100, Jaroslav Bachorik wrote:
> > > > On 1.12.2015 11:17, Severin Gehwolf wrote:
> > > > > On Mon, 2015-11-09 at 10:32 +0100, Severin Gehwolf wrote:
> > > > > > On Wed, 2015-11-04 at 11:54 +0100, Severin Gehwolf wrote:
> > > > > > > Hi,
> > > > > > > 
> > > > > > > Updated webrev with jtreg test in Java:
> > > > > > > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/
> > > > > > > 02/
> > > > > > > bug: https://bugs.openjdk.java.net/browse/JDK-6425769
> > > > > > > 
> > > > > > > I believe this updated webrev addresses all concerns and
> > > > > > > incorporates
> > > > > > > suggestions brought up by Jaroslav and Daniel.
> > > > > > > 
> > > > > > > I'm still looking for a sponsor and a
> > > > > > > hotspot/servicability-
> > > > > > > dev
> > > > > > > reviewer. Could somebody maintaining javax.rmi.ssl have a
> > > > > > > look at
> > > > > > > this
> > > > > > > as well? Thank you!
> > > > > > 
> > > > > > Ping? Friendly reminder that I still need reviewers and a
> > > > > > sponsor for
> > > > > > this.
> > > > > 
> > > > > Anyone?
> > > > 
> > > > I'm sorry for not spotting this earlier:
> > > > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/03.no-
> > > > rmi-
> > > > ssl-factory-
> > > > changes/jdk/src/java.management/share/classes/sun/management/jm
> > > > xrem
> > > > ote/ConnectorBootstrap.java.sdiff.html
> > > > * L442 - the log would contain
> > > > 'com.sun.management.jmxremote.host
> > > > =
> > > > null' if host is not specified; might be better not to print
> > > > this
> > > > out at all
> > > 
> > > Updated webrev which does not print
> > > 'com.sun.management.jmxremote.host = null' if unset:
> > > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/05/
> > > 
> > > > Other than that the change looks good to me. If no one else is
> > > > volunteering I may sponsor this change.
> > > 
> > > Thank you!
> > 
> > Jaroslav, are you still willing to sponsor this? There hasn't been
> > much
> > movement :-/
> 
> Sure. I need to start the compatibility review process before 
> integration and it might take some time.

OK. Thanks for the heads-up!

Cheers,
Severin

> -JB-
> 
> > 
> > Thanks,
> > Severin
> > 
>