Uploaded image for project: 'TorqueBox'
  1. TorqueBox
  2. TORQUE-755

mod_cluster should be easier to setup

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.1
    • Component/s: None
    • Labels:
      None

      Description

      It has become a little difficult to hook TorqueBox up with mod_cluster. This could be improved with two enhancements/bug-fixes:

      1) add instance-id="$

      {jboss.node.name}

      " to the web subsystem element
      2) add advertise-security-key="secret" to the modcluster subsystem element.

      I think #1 should be added by default. And I would like to see #2 added by default, but I could see why it would not be for security reasons.

      For #2 it might make more sense to have a setup script that modified standalone-ha.xml (the way that the java_to_ruby.rb script did for openshift). But this script could even accept a config file, or maybe prompt the user for some values. I'll probably write that either way.

        Gliffy Diagrams

          Activity

          codefinger Joe Kutner created issue -
          Show
          codefinger Joe Kutner added a comment - Related: https://community.jboss.org/message/727166?tstart=0
          tcrawley Toby Crawley made changes -
          Field Original Value New Value
          Fix Version/s 2.0.1 [ 12319251 ]
          benbrowning Benjamin Browning made changes -
          Assignee Benjamin Browning [ benbrowning ]
          benbrowning Benjamin Browning made changes -
          Status Open [ 1 ] Coding In Progress [ 3 ]
          Hide
          benbrowning Benjamin Browning added a comment -

          Fixed by https://github.com/torquebox/torquebox/commit/0a2954340da673ed610b971fec28638c54c49183
          https://github.com/torquebox/torquebox/commit/4dac1f813ca14b9c211e28bd8043a6dbb372854a
          https://github.com/torquebox/torquebox/commit/f6e2c03cacefc525ec302aa03cfc20df39110d65

          Instead of providing default values for instance-id and advertise-security-key, I was able to backport some fixes from AS 7 master (that should land in 7.1.2) to not required these values to bet set, preserving the behavior present in TorqueBox 2.0.0.cr1.

          Show
          benbrowning Benjamin Browning added a comment - Fixed by https://github.com/torquebox/torquebox/commit/0a2954340da673ed610b971fec28638c54c49183 https://github.com/torquebox/torquebox/commit/4dac1f813ca14b9c211e28bd8043a6dbb372854a https://github.com/torquebox/torquebox/commit/f6e2c03cacefc525ec302aa03cfc20df39110d65 Instead of providing default values for instance-id and advertise-security-key, I was able to backport some fixes from AS 7 master (that should land in 7.1.2) to not required these values to bet set, preserving the behavior present in TorqueBox 2.0.0.cr1.
          benbrowning Benjamin Browning made changes -
          Status Coding In Progress [ 3 ] Resolved [ 5 ]
          Resolution Done [ 1 ]
          benbrowning Benjamin Browning made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              benbrowning Benjamin Browning
              Reporter:
              codefinger Joe Kutner
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development