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. Change-Id: I0c39ac88e82f2e511f0025f18586e7a251a17321 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=982065 Signed-off-by: pkliczewski <[email protected]> Signed-off-by: Vinzenz Feenstra <[email protected]> Reviewed-on: http://gerrit.ovirt.org/21506 Reviewed-by: Yaniv Bronhaim <[email protected]> Reviewed-by: Sergey Gotliv <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/vdscli.py.in M tests/Makefile.am A tests/transportWrapperTests.py 3 files changed, 55 insertions(+), 3 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, but someone else must approve Vinzenz Feenstra: Verified Sergey Gotliv: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21506 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c39ac88e82f2e511f0025f18586e7a251a17321 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Sergey Gotliv <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
