Change in ovirt-engine[master]: restapi: Improve logging of exceptions

juan.hernandez at redhat.com juan.hernandez at redhat.com
Thu Feb 5 16:33:13 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Improve logging of exceptions
......................................................................


restapi: Improve logging of exceptions

Currently when an exception is detected while rendering an document the
details are swallowed by the REST framework and lost. This complicates
debugging. To avoid that this patch changes the exception handler so
that it will send the details of the exception to the log.

Change-Id: I8085ae6d09ab49f1706b6c38d5b3e79b712f0241
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/IOExceptionMapper.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/JsonExceptionMapper.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/MalformedIdExceptionMapper.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/MappingExceptionMapper.java
4 files changed, 52 insertions(+), 1 deletion(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved
  Martin Mucha: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8085ae6d09ab49f1706b6c38d5b3e79b712f0241
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list