Dan Kenigsberg has submitted this change and it was merged. Change subject: Catch libvirt timeout and return an error on setVmTicket ......................................................................
Catch libvirt timeout and return an error on setVmTicket Return an errCode instead of an exception libvirt timeout in setVmTicket verb and setting the 'message' to be the exception string representation so upper layers would have more info about what went wrong. Change-Id: Ibb48a1baff1e059f48f71f600f4bf5d9f40a7d11 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=878064 Signed-off-by: Roy Golan <[email protected]> --- M vdsm/libvirtvm.py 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Roy Golan: Verified -- To view, visit http://gerrit.ovirt.org/10285 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb48a1baff1e059f48f71f600f4bf5d9f40a7d11 Gerrit-PatchSet: 12 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Roy Golan <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Peter V. Saveliev <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Shu Ming <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
