TorqueBox
  1. TorqueBox
  2. TORQUE-755

mod_cluster should be easier to setup

    Details

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

      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.

        Activity

        Show
        Joe Kutner
        added a comment - Related: https://community.jboss.org/message/727166?tstart=0
        Toby Crawley
        made changes -
        Field Original Value New Value
        Fix Version/s 2.0.1 [ 12319251 ]
        Benjamin Browning
        made changes -
        Assignee Benjamin Browning [ benbrowning ]
        Benjamin Browning
        made changes -
        Status Open [ 1 ] Coding In Progress [ 3 ]
        Ben Browning <bbrownin@redhat.com> committed 0a2954340da673ed610b971fec28638c54c49183 (1 file)
        Reviews: none

        Provide default value for mod-cluster-config advertise-security-key (TORQUE-755)

        Ben Browning <bbrownin@redhat.com> committed 4dac1f813ca14b9c211e28bd8043a6dbb372854a (3 files)
        Reviews: none

        Ensure nodes without an instance-id set use a UUID in mod_cluster (TORQUE-755)
        This is a hack to essentially backport the fix for JBPAPP-8451 in
        commit
        https://github.com/ctomc/jboss-as/commit/167f41f06e829f76dddacb55bf4bcb11e35795f0
        to TorqueBox so we can release with this issue fixed before AS 7.1.2
        comes out. Once 7.1.2 is out, this commit can be reverted.

        Hide
        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
        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.
        Benjamin Browning
        made changes -
        Status Coding In Progress [ 3 ] Resolved [ 5 ]
        Resolution Done [ 1 ]
        Benjamin Browning
        made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: