changeset 21298:d8a11a962ee3

8024071: In ManagementAgent.start it should be possible to set the jdp.name parameter. Summary: Pass one more property from Agent to JdpController Reviewed-by: jbachorik, sla
author dsamersoff
date Sat, 19 Oct 2013 20:59:42 +0400
parents 813b7d784410
children b4e7e588552d
files jdk/src/share/classes/sun/management/Agent.java jdk/test/sun/management/jdp/JdpTest.sh jdk/test/sun/management/jmxremote/startstop/JMXStartStopTest.sh
diffstat 3 files changed, 7 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/jdk/src/share/classes/sun/management/Agent.java	Sat Oct 19 17:53:44 2013 +0100
+++ b/jdk/src/share/classes/sun/management/Agent.java	Sat Oct 19 20:59:42 2013 +0400
@@ -323,7 +323,7 @@
                     : String.format(
                     "service:jmx:rmi:///jndi/rmi://%s:%s/jmxrmi", hostname, jmxremotePort);
 
-            String instanceName = System.getProperty("com.sun.management.jdp.name");
+            String instanceName = props.getProperty("com.sun.management.jdp.name");
 
             try{
                JdpController.startDiscoveryService(address, port, instanceName, jmxUrlStr);
--- a/jdk/test/sun/management/jdp/JdpTest.sh	Sat Oct 19 17:53:44 2013 +0100
+++ b/jdk/test/sun/management/jdp/JdpTest.sh	Sat Oct 19 20:59:42 2013 +0400
@@ -213,6 +213,7 @@
                 jdp.port=${_port} \
                 jdp.address=${_ip} \
                 jdp.pause=5 \
+                jdp.name=jcmdtest \
                 jmxremote.port=${_jmxport} \
                 jmxremote.authenticate=false \
                 jmxremote.ssl=false
--- a/jdk/test/sun/management/jmxremote/startstop/JMXStartStopTest.sh	Sat Oct 19 17:53:44 2013 +0100
+++ b/jdk/test/sun/management/jmxremote/startstop/JMXStartStopTest.sh	Sat Oct 19 20:59:42 2013 +0400
@@ -551,6 +551,11 @@
  esac 
 done
 
+if [ "${COMPILEJAVA}" = "" ]
+then 
+  COMPILEJAVA=${TESTJAVA}
+fi
+
 if [ ${_compile} = "yes" ]
 then
  _compile