Dan Kenigsberg has submitted this change and it was merged.

Change subject: xmlrpc: Parsing error logging enhancement - vdsClient
......................................................................


xmlrpc: Parsing error logging enhancement - vdsClient

Logging enhancement which helps understand the issues during
parsing xml response obtained from vdsm. Parsing
issues can occur when bad characters are in xml etc.

In order to get raw response we need to wrap one of transport
classes (plain or provided class for ssl) and present it to
the user when ExpatError is raised.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=982065
Change-Id: Ife29c4f7749b9cd8a4ad892f486d91509e505ae4
Signed-off-by: pkliczewski <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/20627
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M lib/vdsm/vdscli.py.in
M tests/Makefile.am
A tests/transportWrapperTests.py
3 files changed, 59 insertions(+), 3 deletions(-)

Approvals:
  Piotr Kliczewski: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ife29c4f7749b9cd8a4ad892f486d91509e505ae4
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: mooli tayer <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to