summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZuul <zuul@review.opendev.org>2021-04-28 21:24:43 +0000
committerGerrit Code Review <review@openstack.org>2021-04-28 21:24:43 +0000
commit10df17638a1587f740c46a574c923df9348c3344 (patch)
treee759450918545837ace8018c1bceb2277c10c187
parent779596b5d8f0117be3e08ea1d5955ef6a03351f0 (diff)
parent06b8f1467dc80203098de983cfc335b898440573 (diff)
downloadnova-20.6.1.tar.gz
Merge "libvirt: 'video.vram' property must be an integer" into stable/traintrain-em20.6.1
-rw-r--r--nova/tests/unit/virt/libvirt/test_driver.py16
-rw-r--r--nova/virt/libvirt/driver.py2
2 files changed, 11 insertions, 7 deletions
diff --git a/nova/tests/unit/virt/libvirt/test_driver.py b/nova/tests/unit/virt/libvirt/test_driver.py
index 4fec9db386..dc613e0fe9 100644
--- a/nova/tests/unit/virt/libvirt/test_driver.py
+++ b/nova/tests/unit/virt/libvirt/test_driver.py
@@ -6100,7 +6100,7 @@ class LibvirtConnTestCase(test.NoDBTestCase,
self.assertEqual(cfg.devices[4].type, "spice")
self.assertEqual(cfg.devices[5].type, "qxl")
- self.assertEqual(cfg.devices[5].vram, 64 * units.Mi / units.Ki)
+ self.assertEqual(cfg.devices[5].vram, 65536)
def _test_add_video_driver(self, model):
self.flags(virt_type='kvm', group='libvirt')
@@ -6111,15 +6111,19 @@ class LibvirtConnTestCase(test.NoDBTestCase,
drvr = libvirt_driver.LibvirtDriver(fake.FakeVirtAPI(), True)
guest = vconfig.LibvirtConfigGuest()
- instance_ref = objects.Instance(**self.test_instance)
- flavor = instance_ref.get_flavor()
+ flavor = objects.Flavor(
+ extra_specs={'hw_video:ram_max_mb': '512'})
image_meta = objects.ImageMeta.from_dict({
- 'properties': {'hw_video_model': model}})
+ 'properties': {
+ 'hw_video_model': model,
+ 'hw_video_ram': 8,
+ },
+ })
self.assertTrue(drvr._guest_add_video_device(guest))
- video = drvr._add_video_driver(guest, image_meta,
- flavor)
+ video = drvr._add_video_driver(guest, image_meta, flavor)
self.assertEqual(model, video.type)
+ self.assertEqual(8192, video.vram) # should be in bytes
def test__add_video_driver(self):
self._test_add_video_driver('qxl')
diff --git a/nova/virt/libvirt/driver.py b/nova/virt/libvirt/driver.py
index aead8470b2..f51463f8fe 100644
--- a/nova/virt/libvirt/driver.py
+++ b/nova/virt/libvirt/driver.py
@@ -5236,7 +5236,7 @@ class LibvirtDriver(driver.ComputeDriver):
raise exception.RequestedVRamTooHigh(req_vram=video_ram,
max_vram=max_vram)
if max_vram and video_ram:
- video.vram = video_ram * units.Mi / units.Ki
+ video.vram = video_ram * units.Mi // units.Ki
guest.add_device(video)
# NOTE(sean-k-mooney): return the video device we added