Uploaded image for project: 'ActiveMQ Artemis'
  1. ActiveMQ Artemis
  2. ARTEMIS-82

start script limitations

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.0.0
    • None
    • None
    • UNIX

    Description

      This issue concerns the start script:
      ./distribution/activemq/src/main/resources/bin/activemq

      There are various issues with JAVA_ARGS at the bottom:

      a) memory parameters are hard-coded in the script and can't be overridden without modifying the script

      b) data.dir is hardcoded in the script and can't be overridden without modifying the script

      Problem (a) was also present in HornetQ while problem (b) was not there.

      With HornetQ 2.3, it was possible to override the data.dir and also add other properties or JMXetric to the JVM command line by setting the CLUSTER_PROPS variable in a local wrapper script. However, setting heap parameters in CLUSTER_PROPS was ineffective because the HornetQ run.sh would then set them to other values later on the command line.

      JBoss / Wildfly seems to do a better job of splitting the logic and configuration, they have a script called bin/standalone.sh that contains the logic and a separate file, bin/standalone.conf that can be customized with environment variables. I think this would be a good model for ActiveMQ 6 to follow. Is there already any discussion about refactoring the script?

      This is important for various reasons:

      • people deploying the broker in production environments don't like to modify the scripts supplied by a vendor, they usually prefer to have a configuration file of their own
      • when upgrading, it is an extra hassle to merge changes into the scripts if they have been modified locally
      • some people may want to run multiple instances using the same installation directory, same scripts and just using different data and log directories

      Attachments

        Issue Links

          Activity

            People

              martyntaylor Martyn Taylor
              daniel.pocock Daniel Pocock
              Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: