Yair Zaslavsky has uploaded a new change for review.

Change subject: extmgr: Engine should start even if there is a wrong 
configuration of extension
......................................................................

extmgr: Engine should start even if there is a wrong configuration of extension

Change-Id: Iac6b522eb8d5fc1f3993780e6c1f09f7e5046757
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1117397
Signed-off-by: Yair Zaslavsky <yzasl...@redhat.com>
---
M 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/extensionsmgr/EngineExtensionsManager.java
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/28/29728/1

diff --git 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/extensionsmgr/EngineExtensionsManager.java
 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/extensionsmgr/EngineExtensionsManager.java
index 07507f9..e01cc6d 100644
--- 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/extensionsmgr/EngineExtensionsManager.java
+++ 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/extensionsmgr/EngineExtensionsManager.java
@@ -79,7 +79,11 @@
                     sort(files);
                     for (File file : files) {
                         if (file.getName().endsWith(".properties")) {
-                            load(file);
+                            try {
+                                load(file);
+                            } catch (Exception ex) {
+                                log.error("Could not load extension based on 
configuration file '{}'. Please check the configuration file is valid.", 
file.getAbsolutePath());
+                            }
                         }
                     }
                 }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iac6b522eb8d5fc1f3993780e6c1f09f7e5046757
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