You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by GitBox <gi...@apache.org> on 2020/12/15 16:37:55 UTC

[GitHub] [karaf] jbonofre opened a new pull request #1267: [KARAF-6917] Remove -Dcom.sun.management.jmxremote by default

jbonofre opened a new pull request #1267:
URL: https://github.com/apache/karaf/pull/1267


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [karaf] jbonofre merged pull request #1267: [KARAF-6917] Remove -Dcom.sun.management.jmxremote by default

Posted by GitBox <gi...@apache.org>.
jbonofre merged pull request #1267:
URL: https://github.com/apache/karaf/pull/1267


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [karaf] coheigea commented on a change in pull request #1267: [KARAF-6917] Remove -Dcom.sun.management.jmxremote by default

Posted by GitBox <gi...@apache.org>.
coheigea commented on a change in pull request #1267:
URL: https://github.com/apache/karaf/pull/1267#discussion_r544902581



##########
File path: manual/src/main/asciidoc/user-guide/monitoring.adoc
##########
@@ -32,14 +32,6 @@ The JMX URL to use by default is:
 service:jmx:rmi:///jndi/rmi://localhost:1099/karaf-root
 ----
 
-If you don't need the remote JMX at all, users can remove
-
-----
--Dcom.sun.management.jmxremote
-----
-
-from bin/karaf|bin/karaf.bat to avoid opening the RMI listening port.
-

Review comment:
       Probably here we need to add documentation to put in -Dcom.sun.management.jmxremote if remote monitoring is needed?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [karaf] jbonofre commented on a change in pull request #1267: [KARAF-6917] Remove -Dcom.sun.management.jmxremote by default

Posted by GitBox <gi...@apache.org>.
jbonofre commented on a change in pull request #1267:
URL: https://github.com/apache/karaf/pull/1267#discussion_r545211775



##########
File path: manual/src/main/asciidoc/user-guide/monitoring.adoc
##########
@@ -32,14 +32,6 @@ The JMX URL to use by default is:
 service:jmx:rmi:///jndi/rmi://localhost:1099/karaf-root
 ----
 
-If you don't need the remote JMX at all, users can remove
-
-----
--Dcom.sun.management.jmxremote
-----
-
-from bin/karaf|bin/karaf.bat to avoid opening the RMI listening port.
-

Review comment:
       We can do that anyway, can't we ? jconsole, decanter JMX, etc, can still connect for remote monitoring. 




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org