[Engine-patches] Change in ovirt-engine[master]: extmgr: add nested exception message into new exception

[Alon Bar-Lev]

Alon Bar-Lev has uploaded a new change for review.

Change subject: extmgr: add nested exception message into new exception
......................................................................

extmgr: add nested exception message into new exception

this enables to see helpful errors while not in debug mode.

Topic: AAA
Change-Id: If321921682d4de599db202956babb1af8a7b315b
Signed-off-by: Alon Bar-Lev <alo...@redhat.com>
---
M 
backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java
1 file changed, 5 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/03/33303/1

diff --git 
a/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java
 
b/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java
index 7d2c3e5..8dfe54a 100644
--- 
a/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java
+++ 
b/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java
@@ -61,7 +61,7 @@
                 }
                 return module;
             } catch (ModuleLoadException exception) {
-                throw new ConfigurationException(String.format("The module 
'%1$s' cannot be loaded.", moduleSpec),
+                throw new ConfigurationException(String.format("The module 
'%1$s' cannot be loaded: %2$s", moduleSpec, exception.getMessage()),
                         exception);
             }
         }
@@ -146,8 +146,8 @@
             props.load(inputStream);
             return loadImpl(props, file);
         } catch (IOException exception) {
-            throw new ConfigurationException(String.format("Can't load object 
configuration file '%1$s'",
-                    file.getAbsolutePath()), exception);
+            throw new ConfigurationException(String.format("Can't load object 
configuration file '%1$s': %2$s",
+                    file.getAbsolutePath(), exception.getMessage()), 
exception);
         }
     }
 
@@ -247,7 +247,7 @@
                );
            }
         } catch (Exception e) {
-            throw new RuntimeException(String.format("Error loading extension 
%1$s", entry.name), e);
+            throw new RuntimeException(String.format("Error loading extension 
'%1$s': %2$s", entry.name, e.getMessage()), e);
         }
         loadedEntries.put(entry.name, entry);
         dumpConfig(entry.extension);
@@ -313,7 +313,7 @@
                     );
             logger.info("Extension '{}' initialized", entry.name);
         } catch (Exception ex) {
-            log.error("Error in activating extension {}. Exception message is 
{}", entry.name, ex.getMessage());
+            log.error("Error in activating extension '{}': {}", entry.name, 
ex.getMessage());
             if (log.isDebugEnabled()) {
                 log.debug(ex.toString(), ex);
             }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If321921682d4de599db202956babb1af8a7b315b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches