x/python-ganttclient
Code Issues Proposed changes

Make libvirt driver default to virtio for KVM/QEMU NICs

Browse Source 
Currently users of KVM/QEMU must set libvirt_use_virtio_for_bridges
to true to get a virtio based NIC, instead of the awful rtl8139
that QEMU defaults to.

Switch libvirt_use_virtio_for_bridges to default to enabled, but
ensure it only takes effect if libvirt_type is 'kvm' or 'qemu'.

Although it is possible that some guest OS do not support virtio
drivers, currently Nova hardcodes use of 'virtio' for disks, so
this change doesn't make the current guest OS support situation
worse.

Obviously in the future, both NIC & disk models will have to be
made fully configurable

Change-Id: I7c0d306921f554ac2d06b5bdb1f9017ab7f9c643
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
This commit is contained in:
Daniel P. Berrange
2012-12-17 15:04:18 +00:00
parent e514dff1d2
commit bff45b4d55
1 changed files with 76 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

76
nova/tests/test_libvirt_vif.py
View File

@@ -120,6 +120,82 @@ class LibvirtVifTestCase(test.TestCase):
self.assertEqual(nic['target_dev'],
node.find("target").get("dev"))
def test_model_novirtio(self):
self.flags(libvirt_use_virtio_for_bridges=False,
libvirt_type='kvm')
d = vif.LibvirtBridgeDriver()
xml = self._get_instance_xml(d)
doc = etree.fromstring(xml)
ret = doc.findall('./devices/interface')
self.assertEqual(len(ret), 1)
node = ret[0]
ret = node.findall("model")
self.assertEqual(len(ret), 0)
ret = node.findall("driver")
self.assertEqual(len(ret), 0)
d.unplug(None, (self.net, self.mapping))
def test_model_kvm(self):
self.flags(libvirt_use_virtio_for_bridges=True,
libvirt_type='kvm')
d = vif.LibvirtBridgeDriver()
xml = self._get_instance_xml(d)
doc = etree.fromstring(xml)
ret = doc.findall('./devices/interface')
self.assertEqual(len(ret), 1)
node = ret[0]
model = node.find("model").get("type")
self.assertEqual(model, "virtio")
ret = node.findall("driver")
self.assertEqual(len(ret), 0)
d.unplug(None, (self.net, self.mapping))
def test_model_qemu(self):
self.flags(libvirt_use_virtio_for_bridges=True,
libvirt_type='qemu')
d = vif.LibvirtBridgeDriver()
xml = self._get_instance_xml(d)
doc = etree.fromstring(xml)
ret = doc.findall('./devices/interface')
self.assertEqual(len(ret), 1)
node = ret[0]
model = node.find("model").get("type")
self.assertEqual(model, "virtio")
driver = node.find("driver").get("name")
self.assertEqual(driver, "qemu")
d.unplug(None, (self.net, self.mapping))
def test_model_xen(self):
self.flags(libvirt_use_virtio_for_bridges=True,
libvirt_type='xen')
d = vif.LibvirtBridgeDriver()
xml = self._get_instance_xml(d)
doc = etree.fromstring(xml)
ret = doc.findall('./devices/interface')
self.assertEqual(len(ret), 1)
node = ret[0]
ret = node.findall("model")
self.assertEqual(len(ret), 0)
ret = node.findall("driver")
self.assertEqual(len(ret), 0)
d.unplug(None, (self.net, self.mapping))
def test_bridge_driver(self):
d = vif.LibvirtBridgeDriver()
xml = self._get_instance_xml(d)