From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has posted comments on this change.
Change subject: vm: create a vnic with vhostuser backend ......................................................................
Patch Set 31: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/74327/31/lib/vdsm/virt/vmdevices/network.py File lib/vdsm/virt/vmdevices/network.py:
Line 303: detach_detachable(self.hostdev) Line 304: else: Line 305: bridge_info = supervdsm.getProxy().ovs_bridge(self.network) Line 306: if bridge_info and bridge_info['dpdk_enabled']: Line 307: self._is_vhostuser = True this is not called during recovery flow. We should either persist this in conf, or pass it to __init__ during recovery, or recalculate during deardown+getXML. Line 308: self._create_vhost_port(bridge_info['name']) Line 309: Line 310: def teardown(self): Line 311: if self.is_hostdevice:
vdsm-patches@lists.fedorahosted.org