]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
GlusterFS: Delete active snapshot file on volume delete.
authorThang Pham <thang.g.pham@gmail.com>
Tue, 8 Apr 2014 20:45:34 +0000 (16:45 -0400)
committerThang Pham <thang.g.pham@gmail.com>
Wed, 9 Apr 2014 16:45:21 +0000 (12:45 -0400)
commit5f00cad02da1093d71f636add0810a538cbd444f
treef9f561224fb09cb2ad209e5112774428b8c33c32
parent1d1a9b3fe7bc4acd19c68b58791f34389148bb9d
GlusterFS: Delete active snapshot file on volume delete.

If a snapshot is taken of a volume that is attached to an active
instance, the volume file used by the instance will be switched to
the new snapshot file that is created.  When you delete the
snapshot, the base volume file will be merged with the snapshot
file and the base volume is deleted.  Upon a deleting the active
volume, the active snapshot file is not deleted because it does not
have the expected name that cinder is looking for, i.e.
volume-<uuid>.  Instead, the snapshot file has the name
volume-<uuid>.<snapshot-uuid>.  This patch looks at the volume info
file to find any active snapshot file and properly delete it when
the volume is deleted.

Change-Id: Ib0af4401d839ec3bd1eb3a81e1671811e0d4a288
Closes-Bug: #1300303
cinder/tests/test_glusterfs.py
cinder/volume/drivers/glusterfs.py