Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm: Fix check for connection issues with libvirt ......................................................................
vdsm: Fix check for connection issues with libvirt There has been a made a mistake by mixing domain codes with error codes of libvirt. Additionally libvirt is reporting VIR_ERR_INTERNAL_ERROR if libvirtd gets restarted or stopped somehow (e.g. after a crash) This fix takes care to cleanup the mixup and adds the missing error code. Change-Id: I051d007529e22b0c4ca703042ce1601b4b2817e0 Signed-off-by: Vinzenz Feenstra <[email protected]> Bug-Url: https://bugzilla.redhat.com/852956 --- M lib/vdsm/libvirtconnection.py 1 file changed, 11 insertions(+), 5 deletions(-) Approvals: Saggi Mizrahi: Looks good to me, but someone else must approve Vinzenz Feenstra: Verified Dan Kenigsberg: Looks good to me, approved Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13990 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I051d007529e22b0c4ca703042ce1601b4b2817e0 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
