Dan Kenigsberg has submitted this change and it was merged. Change subject: BZ#840594 Fix netinfo when libvirt bridge does not exist. ......................................................................
BZ#840594 Fix netinfo when libvirt bridge does not exist. In the case where a bridge that is configured in libvirt is manually removed or renamed, netinfo throws an exception when trying to discover which ports are assigned to the bridge. This patch catches the OSError exception of trying to list the ports of a non- existing bridge interface, effectively filtering the non-existing interface from the network capabilities. Also adds a unittest for getting network information of non-existing bridges. Change-Id: I2538a8ce2fafd8efed834b87b20a63d058afd96b Signed-off-by: Antoni S. Puimedon <asegu...@redhat.com> --- M tests/netinfoTests.py M vdsm/netinfo.py 2 files changed, 23 insertions(+), 3 deletions(-) Approvals: Dan Kenigsberg: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7127 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2538a8ce2fafd8efed834b87b20a63d058afd96b Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <asegu...@redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Igor Lvovsky <ilvov...@redhat.com> Gerrit-Reviewer: Livnat Peer <lp...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches