[jira] Resolved: (GERONIMO-3833) Hard-coded gbean names and versions in monitoring code

["Viet Hung Nguyen (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/GERONIMO-3833?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Viet Hung Nguyen resolved GERONIMO-3833.
----------------------------------------

    Resolution: Fixed

> Hard-coded gbean names and versions in monitoring code
> ------------------------------------------------------
>
>                 Key: GERONIMO-3833
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-3833
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: monitoring
>    Affects Versions: 2.1, 2.1.1, 2.2
>            Reporter: Jarek Gawor
>            Assignee: Viet Hung Nguyen
>             Fix For: 2.1.1, 2.2
>
>
> The monitoring code has hard-coded values in Java code and sql file:
> 1)
> The ./plugins/monitoring/mconsole-war/src/main/java/org/apache/geronimo/monitoring/console/MRCConnector.java contains the following constant:
> private static final String PATH = "geronimo:ServiceModule=org.apache.geronimo.plugins.monitoring/agent-car-jmx/2.1-SNAPSHOT/car,J2EEServer=geronimo,
> name=MasterRemoteControlJMX,j2eeType=GBean";
> 2)
> The  ./plugins/monitoring/mconsole-ear/src/main/resources/MonitoringClientDB.sql contains a bunch of the following values:
> 'geronimo:J2EEServer=geronimo,ServiceModule=org.apache.geronimo.configs/tomcat6/2.1/car,j2eeType=GBean,name=TomcatWebConnector'
> I'm not sure how these are used but in general these type of hardcoded values should be avoided. It's really hard to maintain and keep track of.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.