summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2016-12-21 16:03:28 +0000
committerGerrit Code Review <review@openstack.org>2016-12-21 16:03:29 +0000
commitfba6f756afd15cb09742b7a9f237a4e061dcda70 (patch)
tree973e827eda6ea7097aaf60591ae51a4045a689d5
parent60c3ce26b6851ff3869f3a23823a094522df1ca5 (diff)
parentd37b6a7cb055bd245c39c231760822d39c65b45f (diff)
downloadtempest-14.0.0.tar.gz
Merge "Delete volume first before deleting snapshot"14.0.0
-rw-r--r--tempest/api/volume/test_volumes_snapshots.py8
1 files changed, 8 insertions, 0 deletions
diff --git a/tempest/api/volume/test_volumes_snapshots.py b/tempest/api/volume/test_volumes_snapshots.py
index 3c7a2c80f..6f8589167 100644
--- a/tempest/api/volume/test_volumes_snapshots.py
+++ b/tempest/api/volume/test_volumes_snapshots.py
@@ -140,6 +140,14 @@ class VolumesV2SnapshotTestJSON(base.BaseVolumeTest):
# Destination volume bigger than source snapshot
dst_vol = self.create_volume(snapshot_id=src_snap['id'],
size=src_size + 1)
+ # NOTE(zhufl): dst_vol is created based on snapshot, so dst_vol
+ # should be deleted before deleting snapshot, otherwise deleting
+ # snapshot will end with status 'error-deleting'. This depends on
+ # the implementation mechanism of vendors, generally speaking,
+ # some verdors will use "virtual disk clone" which will promote
+ # disk clone speed, and in this situation the "disk clone"
+ # is just a relationship between volume and snapshot.
+ self.addCleanup(self.delete_volume, self.volumes_client, dst_vol['id'])
volume = self.volumes_client.show_volume(dst_vol['id'])['volume']
# Should allow