Federico Simoncelli has uploaded a new change for review.

Change subject: core: log stacktraces for failed VDSCommandBase calls
......................................................................

core: log stacktraces for failed VDSCommandBase calls

Signed-off-by: Federico Simoncelli <[email protected]>
Change-Id: I19d03f42a061531757daf870a6ac8eed34c2f992
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VDSCommandBase.java
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/43/8243/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VDSCommandBase.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VDSCommandBase.java
index 04c0c0d..804ef74 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VDSCommandBase.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VDSCommandBase.java
@@ -113,6 +113,9 @@
 
     private void logException(RuntimeException ex) {
         log.errorFormat("Command {0} execution failed. Exception: {1}", 
getCommandName(), ExceptionUtils.getMessage(ex));
+        if (!(ex instanceof VDSExceptionBase)) {
+            log.error(ExceptionUtils.getStackTrace(ex));
+        }
     }
 
     protected String getAdditionalInformation() {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I19d03f42a061531757daf870a6ac8eed34c2f992
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to