openstack/nova
Code Issues Proposed changes

Drop MIN_LIBVIRT_SET_ADMIN_PASSWD

Browse Source 
The updated minimum required libvirt (1.3.1) and QEMU (2.5.0) for
"Rocky" matches this now, so we can drop the compatibility code.

Also remove / fix relevant unit tests.

Change-Id: Ib298817016515538adf7be46af79e98bcd5df41f
Signed-off-by: Kashyap Chamarthy <kchamart@redhat.com>
This commit is contained in:
Kashyap Chamarthy
2018-04-18 12:58:52 +02:00
parent 403320bc99
commit 53eab6d846
2 changed files with 0 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
nova/virt/libvirt/driver.py
View File

@@ -245,9 +245,6 @@ BAD_LIBVIRT_CPU_POLICY_VERSIONS = [(1, 2, 10)]
# Virtuozzo driver support
MIN_VIRTUOZZO_VERSION = (7, 0, 0)
# Ability to set the user guest password with Qemu
MIN_LIBVIRT_SET_ADMIN_PASSWD = (1, 2, 16)
# Ability to set the user guest password with parallels
MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD = (2, 0, 0)
@@ -2007,9 +2004,6 @@ class LibvirtDriver(driver.ComputeDriver):
MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD):
raise exception.SetAdminPasswdNotSupported()
elif CONF.libvirt.virt_type in ('kvm', 'qemu'):
if not self._host.has_min_version(
MIN_LIBVIRT_SET_ADMIN_PASSWD):
raise exception.SetAdminPasswdNotSupported()
if not image_meta.properties.get('hw_qemu_guest_agent', False):
raise exception.QemuGuestAgentNotEnabled()
else: