Yair Zaslavsky has uploaded a new change for review.

Change subject: aaa: Adding the ability to enabled/disable an extension via 
engine configration file
......................................................................

aaa: Adding the ability to enabled/disable an extension via engine configration 
file

Change-Id: I817d58f16fa32d37379a9f98d697538df75c28a6
Topic: AAA
Signed-off-by: Yair Zaslavsky <yzasl...@redhat.com>
---
M 
backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionManager.java
1 file changed, 9 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/77/24977/1

diff --git 
a/backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionManager.java
 
b/backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionManager.java
index 25d9c61..8bde0e8 100644
--- 
a/backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionManager.java
+++ 
b/backend/manager/modules/extension-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionManager.java
@@ -115,8 +115,8 @@
         return result;
     }
 
-    public void load(EngineLocalConfig config) throws ConfigurationException {
-        for (File directory : config.getExtensionsDirectories()) {
+    public void load() throws ConfigurationException {
+        for (File directory : 
EngineLocalConfig.getInstance().getExtensionsDirectories()) {
             load(directory);
         }
         activate();
@@ -204,6 +204,13 @@
      */
     private void activate() {
         for (ExtensionEntry entry : loadedEntries.values()) {
+            //Engine local config might override the enabled property of the 
configuration
+            //if a proper ENGINE_EXTENSION_ENABLE_<NAME> entry exists at the 
engine config.
+            entry.enabled =
+                    EngineLocalConfig.getInstance().getBoolean(
+                            "ENGINE_EXTENSION_ENABLE_" + entry.getName(),
+                            entry.enabled
+                    );
             if (entry.enabled) {
                 try {
                     entry.extension = (Extension) lookupService(


-- 
To view, visit http://gerrit.ovirt.org/24977
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I817d58f16fa32d37379a9f98d697538df75c28a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to