Edward Haas has uploaded a new change for review.

Change subject: virt net: Interface default driver name set to 'vhost'
......................................................................

virt net: Interface default driver name set to 'vhost'

When following these steps:
- Create a VM with vnic_profile that has queues custom properties.
- Start the VM.
- Hot-unplug the vNIC from the VM.

An exception occurs:
libvirtError: unsupported configuration:
    Unknown interface <driver name=''> has been specified
    
Due to a missing default driver name.


Change-Id: I508675f9dd40ba70803938902a1c2f8fced00504
Bug-Url: https://bugzilla.redhat.com/1359520
Signed-off-by: Edward Haas <edwa...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/61297
Continuous-Integration: Jenkins CI
Tested-by: Meni Yakove <myak...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
Reviewed-by: Petr Horáček <phora...@redhat.com>
(cherry picked from commit 00308a4a9a4c666397655cc749e93a019b87e4dd)
---
M vdsm/virt/vmdevices/network.py
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/72/61772/1

diff --git a/vdsm/virt/vmdevices/network.py b/vdsm/virt/vmdevices/network.py
index 755c794..16321d0 100644
--- a/vdsm/virt/vmdevices/network.py
+++ b/vdsm/virt/vmdevices/network.py
@@ -64,6 +64,9 @@
             self.driver['queues'] = self.custom['queues']
         except KeyError:
             pass    # interface queues not specified
+        else:
+            if 'name' not in self.driver:
+                self.driver['name'] = 'vhost'
 
         try:
             self.sndbufParam = self.conf['custom']['sndbuf']


-- 
To view, visit https://gerrit.ovirt.org/61772
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I508675f9dd40ba70803938902a1c2f8fced00504
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Edward Haas <edwa...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to