Geronimo
  1. Geronimo
  2. GERONIMO-4688

hardcode port in plan.xml of plugin farming.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: 2.1.4, 2.1.5, 2.2
    • Fix Version/s: 2.2.2, 3.0.0
    • Component/s: Clustering
    • Security Level: public (Regular issues)
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      In Gtrunk\plugins\clustering\farming\src\main\plan\plan.xml, there are some hardcodeed ser/pass/host/port string in it. I'm not sure if they should be something like $

      {PlanClusterNodeName} so that they can be set in server\config\config_substitution.properties.



      <gbean name="NodeInfo" class="org.apache.geronimo.farm.config.BasicNodeInfo">
      <attribute name="name">${PlanClusterNodeName}

      </attribute>
      <xml-attribute name="extendedJMXConnectorInfo">
      <ns:javabean xmlns:ns="http://geronimo.apache.org/xml/ns/deployment/javabean-1.0" class="org.apache.geronimo.farm.deployment.DeploymentExtendedJMXConnectorInfo">
      <ns:property name="username">system</ns:property>
      <ns:property name="password">manager</ns:property>
      <ns:property name="protocol">rmi</ns:property>
      <ns:property name="host">localhost</ns:property>
      <ns:property name="port">1099</ns:property>
      <ns:property name="urlPath">JMXConnector</ns:property>
      <ns:property name="local">true</ns:property>
      </ns:javabean>
      </xml-attribute>
      </gbean>

      1. G4688_G21.patch
        3 kB
        Shawn Jiang
      2. G4688_trunk.patch
        3 kB
        Shawn Jiang

        Activity

        Hide
        Shawn Jiang added a comment -

        Uploading the patches to fix this JIRA.

        Show
        Shawn Jiang added a comment - Uploading the patches to fix this JIRA.
        Hide
        Ivan added a comment -

        Commit the changes to trunk At revision: 789212, 2.1.5-snapshot At revision: 789213. Thanks Shawn Jiang for the patch !

        Show
        Ivan added a comment - Commit the changes to trunk At revision: 789212, 2.1.5-snapshot At revision: 789213. Thanks Shawn Jiang for the patch !
        Hide
        Shawn Jiang added a comment -

        Seems the patches caused a server build error. Please revert the commit though I can't recreate the build error.

        Show
        Shawn Jiang added a comment - Seems the patches caused a server build error. Please revert the commit though I can't recreate the build error.
        Hide
        Ivan added a comment -

        I think the build error is caused by the jaxb issue, Geronimo-4668

        Show
        Ivan added a comment - I think the build error is caused by the jaxb issue, Geronimo-4668
        Hide
        David Jencks added a comment -

        AFAICT the patch was committed and works?

        Show
        David Jencks added a comment - AFAICT the patch was committed and works?
        Hide
        Shawn Jiang added a comment -

        The patches was reverted due to a known JAXB marshaller issue. Reopen it to track this problem.

        Show
        Shawn Jiang added a comment - The patches was reverted due to a known JAXB marshaller issue. Reopen it to track this problem.
        Hide
        David Jencks added a comment -

        not fixed in 2.2

        Show
        David Jencks added a comment - not fixed in 2.2
        Hide
        Shawn Jiang added a comment -

        won't fix this in 2.1.5 because the jaxb limitation.

        Show
        Shawn Jiang added a comment - won't fix this in 2.1.5 because the jaxb limitation.
        Hide
        Shawn Jiang added a comment -

        I prefer to change the fix target from 2.1.5 to 2.1.6 after 2.1.5 released.

        Show
        Shawn Jiang added a comment - I prefer to change the fix target from 2.1.5 to 2.1.6 after 2.1.5 released.
        Hide
        Shawn Jiang added a comment -

        Won't fix this in 2.2.1

        Show
        Shawn Jiang added a comment - Won't fix this in 2.2.1
        Hide
        Shawn Jiang added a comment -

        The Jaxb issue was fixed here: https://issues.apache.org/jira/browse/GERONIMO-4861

        I will find some time to update the patch of this JIRA and commit it.

        Show
        Shawn Jiang added a comment - The Jaxb issue was fixed here: https://issues.apache.org/jira/browse/GERONIMO-4861 I will find some time to update the patch of this JIRA and commit it.
        Hide
        Shawn Jiang added a comment -

        Seems it's not a problem for users according to our instruction to setup the farm.

        Show
        Shawn Jiang added a comment - Seems it's not a problem for users according to our instruction to setup the farm.

          People

          • Assignee:
            Shawn Jiang
            Reporter:
            Shawn Jiang
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development