From 99a63a6633f623cc4e6c9b361965a9c0935113cf Mon Sep 17 00:00:00 2001 From: Lee Yarwood Date: Thu, 25 Apr 2019 15:34:41 +0100 Subject: [PATCH] libvirt: Always disconnect volumes after libvirtError exceptions Building on Ib440f4f2e484312af5f393722363846f6c95b760 we should always attempt to disconnect volumes from the host when exceptions are thrown while attempting to attach a volume to a domain. This was previously done for generic exceptions but not for libvirtError exceptions. Closes-Bug: #1826523 Change-Id: If21230869826c992e7d0398434b9a4b255940213 (cherry picked from commit 091a910576d9b580678f1881fffa425ab4632f48) (cherry picked from commit 048d5b790f3da2756a0a1bf2bc015812cb24d53a) (cherry picked from commit 786083c91fa69ba9ff90bafe80d3b83b9f9bbc69) --- nova/tests/unit/virt/libvirt/test_driver.py | 10 ++++++++-- nova/virt/libvirt/driver.py | 2 ++ 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/nova/tests/unit/virt/libvirt/test_driver.py b/nova/tests/unit/virt/libvirt/test_driver.py index 5cafb592348f..d4512ee3f039 100644 --- a/nova/tests/unit/virt/libvirt/test_driver.py +++ b/nova/tests/unit/virt/libvirt/test_driver.py @@ -7281,11 +7281,13 @@ class LibvirtConnTestCase(test.NoDBTestCase, mock.patch.object(drvr._host, 'get_guest'), mock.patch('nova.virt.libvirt.driver.LOG'), mock.patch.object(drvr, '_connect_volume'), + mock.patch.object(drvr, '_disconnect_volume'), mock.patch.object(drvr, '_get_volume_config'), mock.patch.object(drvr, '_check_discard_for_attach_volume'), mock.patch.object(drvr, '_build_device_metadata'), ) as (mock_get_guest, mock_log, mock_connect_volume, - mock_get_volume_config, mock_check_discard, mock_build_metadata): + mock_disconnect_volume, mock_get_volume_config, + mock_check_discard, mock_build_metadata): mock_conf = mock.MagicMock() mock_guest = mock.MagicMock() @@ -7299,6 +7301,7 @@ class LibvirtConnTestCase(test.NoDBTestCase, self.context, connection_info, instance, "/dev/vdb", disk_bus=bdm['disk_bus'], device_type=bdm['device_type']) mock_log.warning.assert_called_once() + mock_disconnect_volume.assert_called_once() @mock.patch('nova.virt.libvirt.blockinfo.get_info_from_bdm') def test_attach_volume_with_libvirt_exception(self, mock_get_info): @@ -7320,11 +7323,13 @@ class LibvirtConnTestCase(test.NoDBTestCase, mock.patch.object(drvr._host, 'get_guest'), mock.patch('nova.virt.libvirt.driver.LOG'), mock.patch.object(drvr, '_connect_volume'), + mock.patch.object(drvr, '_disconnect_volume'), mock.patch.object(drvr, '_get_volume_config'), mock.patch.object(drvr, '_check_discard_for_attach_volume'), mock.patch.object(drvr, '_build_device_metadata'), ) as (mock_get_guest, mock_log, mock_connect_volume, - mock_get_volume_config, mock_check_discard, mock_build_metadata): + mock_disconnect_volume, mock_get_volume_config, + mock_check_discard, mock_build_metadata): mock_conf = mock.MagicMock() mock_guest = mock.MagicMock() @@ -7339,6 +7344,7 @@ class LibvirtConnTestCase(test.NoDBTestCase, disk_bus=bdm['disk_bus'], device_type=bdm['device_type']) mock_log.exception.assert_called_once_with(u'Failed to attach ' 'volume at mountpoint: %s', '/dev/vdb', instance=instance) + mock_disconnect_volume.assert_called_once() @mock.patch('nova.utils.get_image_from_system_metadata') @mock.patch('nova.virt.libvirt.blockinfo.get_info_from_bdm') diff --git a/nova/virt/libvirt/driver.py b/nova/virt/libvirt/driver.py index 42f6ec83d241..44cdc7d53a58 100644 --- a/nova/virt/libvirt/driver.py +++ b/nova/virt/libvirt/driver.py @@ -1499,6 +1499,8 @@ class LibvirtDriver(driver.ComputeDriver): else: LOG.exception(_('Failed to attach volume at mountpoint: ' '%s'), mountpoint, instance=instance) + self._disconnect_volume(context, connection_info, instance, + encryption=encryption) except Exception: LOG.exception(_('Failed to attach volume at mountpoint: %s'), mountpoint, instance=instance)