Change in vdsm[master]: netinfo: added configuration to show dummy NIC's too

danken at redhat.com danken at redhat.com
Tue Oct 30 07:30:08 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: netinfo: added configuration to show dummy NIC's too
......................................................................


Patch Set 2: I would prefer that you didn't submit this

(1 inline comment)

....................................................
File vdsm/netinfo.py
Line 43: 
Line 44: LIBVIRT_NET_PREFIX = 'vdsm-'
Line 45: 
Line 46: 
Line 47: def match_nic_name(nic, patterns):
please note my inline comments.
Line 48:     return any(map(lambda p: fnmatch(nic, p), patterns))
Line 49: 
Line 50: def nics():
Line 51:     res = []


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ica0ad974e6dd0d144a6bb0904fdcd6bbce99ff9f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hercinger Viktor <hercinger.viktor at gmail.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Hercinger Viktor <hercinger.viktor at gmail.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>


More information about the vdsm-patches mailing list