openstack/cinder
Code Issues Proposed changes

Merge "VMware: Fix revert-to-snapshot"

Browse Source
This commit is contained in:
Zuul 2018-09-21 09:20:29 +00:00 committed by Gerrit Code Review
commit 8bd4ff7e6a
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cinder/tests/unit/volume/drivers/vmware/test_vmware_vmdk.py
View File

@ -3240,7 +3240,7 @@ class VMwareVcVmdkDriverTestCase(test.TestCase):
self._driver.revert_to_snapshot(
mock.sentinel.context, volume, snapshot)
vops.get_backing.assert_called_once_with(volume.name)
vops.get_backing.assert_called_once_with(volume.name, volume.id)
vops.revert_to_snapshot.assert_not_called()
@mock.patch.object(VMDK_DRIVER, 'volumeops')
@ -3268,7 +3268,7 @@ class VMwareVcVmdkDriverTestCase(test.TestCase):
self._driver.revert_to_snapshot(
mock.sentinel.context, volume, snapshot)
vops.get_backing.assert_called_once_with(volume.name)
vops.get_backing.assert_called_once_with(volume.name, volume.id)
vops.revert_to_snapshot.assert_called_once_with(backing,
snapshot.name)

2
cinder/volume/drivers/vmware/vmdk.py
View File

@ -2218,7 +2218,7 @@ class VMwareVcVmdkDriver(driver.VolumeDriver):
raise exception.InvalidSnapshot("Cannot revert to template "
"based snapshot")
backing = self.volumeops.get_backing(volume.name)
backing = self.volumeops.get_backing(volume.name, volume.id)
if not backing:
LOG.debug("Backing does not exist for volume.", resource=volume)
else: