Qpid
  1. Qpid
  2. QPID-2581

Create a ConfigurationManager to allow plugins to provide configuration handling classes

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7
    • Component/s: Java Broker
    • Labels:
      None

      Description

      To allow plugins to handle configuration add a ConfigurationManager that will load ConfigurationPlugins during the configuration process before the broker starts up.

      This will allow the configuration values to be processed and determined to be valid input before we actually attempt to continue broker startup.

        Activity

        Hide
        Robbie Gemmell added a comment -

        Resolving, no additional work performed since the patch I reviewed and applied.

        Show
        Robbie Gemmell added a comment - Resolving, no additional work performed since the patch I reviewed and applied.
        Hide
        Robbie Gemmell added a comment -

        Patch applied

        Show
        Robbie Gemmell added a comment - Patch applied
        Hide
        Andrew Kennedy added a comment -

        Changed due to svn rebase

        Show
        Andrew Kennedy added a comment - Changed due to svn rebase
        Hide
        Andrew Kennedy added a comment -

        Fixed

        Show
        Andrew Kennedy added a comment - Fixed
        Hide
        Andrew Kennedy added a comment -

        qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ConfigurationManager.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/QueueConfig.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/QueueConfiguration.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/SecurityConfiguration.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ServerConfiguration.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/VirtualHostConfiguration.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/plugins/ConfigurationPlugin.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/plugins/ConfigurationPluginFactory.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/connection/ConnectionRegistry.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/registry/ApplicationRegistry.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/registry/BrokerConfigAdapter.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/registry/ConfigurationFileApplicationRegistry.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/registry/IApplicationRegistry.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/ManagedVirtualHost.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/VirtualHost.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/VirtualHostConfigRecoveryHandler.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/VirtualHostImpl.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/plugins/VirtualHostPlugin.java
        qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/plugins/VirtualHostPluginFactory.java
        qpid/java/broker/src/test/java/org/apache/qpid/server/configuration/ServerConfigurationTest.java
        qpid/java/broker/src/test/java/org/apache/qpid/server/util/NullApplicationRegistry.java
        qpid/java/broker/src/test/java/org/apache/qpid/server/util/TestApplicationRegistry.java
        qpid/java/systests/src/main/java/org/apache/qpid/server/logging/AlertingTest.java
        qpid/java/systests/src/main/java/org/apache/qpid/server/logging/DerbyMessageStoreLoggingTest.java
        qpid/java/systests/src/main/java/org/apache/qpid/server/logging/MemoryMessageStoreLoggingTest.java
        qpid/java/systests/src/main/java/org/apache/qpid/server/logging/VirtualHostLoggingTest.java

        Show
        Andrew Kennedy added a comment - qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ConfigurationManager.java qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/QueueConfig.java qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/QueueConfiguration.java qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/SecurityConfiguration.java qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ServerConfiguration.java qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/VirtualHostConfiguration.java qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/plugins/ConfigurationPlugin.java qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/plugins/ConfigurationPluginFactory.java qpid/java/broker/src/main/java/org/apache/qpid/server/connection/ConnectionRegistry.java qpid/java/broker/src/main/java/org/apache/qpid/server/registry/ApplicationRegistry.java qpid/java/broker/src/main/java/org/apache/qpid/server/registry/BrokerConfigAdapter.java qpid/java/broker/src/main/java/org/apache/qpid/server/registry/ConfigurationFileApplicationRegistry.java qpid/java/broker/src/main/java/org/apache/qpid/server/registry/IApplicationRegistry.java qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/ManagedVirtualHost.java qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/VirtualHost.java qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/VirtualHostConfigRecoveryHandler.java qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/VirtualHostImpl.java qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/plugins/VirtualHostPlugin.java qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/plugins/VirtualHostPluginFactory.java qpid/java/broker/src/test/java/org/apache/qpid/server/configuration/ServerConfigurationTest.java qpid/java/broker/src/test/java/org/apache/qpid/server/util/NullApplicationRegistry.java qpid/java/broker/src/test/java/org/apache/qpid/server/util/TestApplicationRegistry.java qpid/java/systests/src/main/java/org/apache/qpid/server/logging/AlertingTest.java qpid/java/systests/src/main/java/org/apache/qpid/server/logging/DerbyMessageStoreLoggingTest.java qpid/java/systests/src/main/java/org/apache/qpid/server/logging/MemoryMessageStoreLoggingTest.java qpid/java/systests/src/main/java/org/apache/qpid/server/logging/VirtualHostLoggingTest.java
        Hide
        Andrew Kennedy added a comment -

        Update configuration manager to handle multiple plugins that are responsible for the same piece of configuration

        Show
        Andrew Kennedy added a comment - Update configuration manager to handle multiple plugins that are responsible for the same piece of configuration
        Hide
        Andrew Kennedy added a comment -

        Have added capability to use multiple ConfigurationPlugins that process the same paths. Will either add patch here, or roll-up into the ACL patchset.

        Show
        Andrew Kennedy added a comment - Have added capability to use multiple ConfigurationPlugins that process the same paths. Will either add patch here, or roll-up into the ACL patchset.
        Hide
        Martin Ritchie added a comment -

        ConfigurationPlugins are loaded but currently onyl the first plugin reporting to process a given path will be used. Needs to be updated to allow <n> plugins to be able to process the configuration.

        Show
        Martin Ritchie added a comment - ConfigurationPlugins are loaded but currently onyl the first plugin reporting to process a given path will be used. Needs to be updated to allow <n> plugins to be able to process the configuration.

          People

          • Assignee:
            Robbie Gemmell
            Reporter:
            Martin Ritchie
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development