Change in vdsm[master]: BZ#811807 Enable network filtering

danken at redhat.com danken at redhat.com
Mon Aug 20 22:15:13 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: BZ#811807 Enable network filtering
......................................................................


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

(1 inline comment)

....................................................
File vdsm/libvirtvm.py
Line 963:         if hasattr(self, 'filters'):
Line 964:             for filterName in self.filters:
Line 965:                 m = doc.createElement('filterref')
Line 966:                 m.setAttribute('filter', filterName)
Line 967:                 iface.appendChild(m)
I'm pretty sure we've seen that libvirt (currently?) supports only one filterref per nic, so at the moment the api is misleading.

Either change the API to accept only one, or open an RFE for libvirt to support more. Either way, this issue should be documented in the commit message.
PS I am not sure what would the semantics of multiple filterrefs be - OR? AND? Priority? I'm not sure it is going to fly so easily.
Line 968:         if hasattr(self, 'bootOrder'):
Line 969:             bootOrder = doc.createElement('boot')
Line 970:             bootOrder.setAttribute('order', self.bootOrder)
Line 971:             iface.appendChild(bootOrder)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I33b3dc7021997f4d0c1194049b1c12ac3528f067
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list