Antoni Segura Puimedon has posted comments on this change.
Change subject: Optimize netinfo.networks.
......................................................................
Patch Set 1: (1 inline comment)
Good catch Jarod! What do you think of what I propose in the inline comment?
....................................................
File vdsm/netinfo.py
Line 89: 'red': { 'iface': 'red', 'bridged': False } }
Line 90: """
Line 91: nets = {}
Line 92: conn = libvirtconnection.get()
Line 93: allNets = ((net, net.name()) for net in conn.listAllNetworks(0))
@Jarod.w Maybe rebase on top of http://gerrit.ovirt.org/#/c/13163/
Line 94: vdsmNets = ((net[0],
Line 95: net[1][len(LIBVIRT_NET_PREFIX):])
Line 96: for net in allNets if
net[1].startswith(LIBVIRT_NET_PREFIX))
Line 97: for net, netname in vdsmNets:
--
To view, visit http://gerrit.ovirt.org/13159
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2cd332819363c4399335358686e19d6a1dc3b45e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Jarod.w <work.iec23...@gmail.com>
Gerrit-Reviewer: Mark Wu <wu...@linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
