Antoni Segura Puimedon has posted comments on this change.

Change subject: [WIP] netwiring: [4/4] Add API definitions.
......................................................................


Patch Set 22: (1 inline comment)

....................................................
File vdsm/netinfo.py
Line 76:     return [b.split('/')[-2] for b in 
glob.glob('/sys/class/net/*/bridge')
Line 77:             if b.split('/')[-2] != DUMMY_BRIDGE]
Line 78: 
Line 79: 
Line 80: def networks():
It was being filtered by the fact that it didn't start with LIBVIRT_NET_PREFIX
Line 81:     """
Line 82:     Get dict of networks from libvirt
Line 83: 
Line 84:     :returns: dict of networkname={properties}


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3b9b4f49f80466a83e3e13f1042ac2a8866c6bcd
Gerrit-PatchSet: 22
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: ShaoHe Feng <[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