]> 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)
committerThomas Goirand <thomas@goirand.fr>
Mon, 9 Jun 2014 14:08:50 +0000 (22:08 +0800)
commite14c7d4209355544a9253d60f5fe5372d2dda2a9
treeb0d6b21631c6ee5b3f5262b4c33211af0f73c892
parentd1b8b5f86ff0766284a2bf52712a316fee52a99c
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
(cherry picked from commit 5f00cad02da1093d71f636add0810a538cbd444f)
cinder/tests/test_glusterfs.py
cinder/volume/drivers/glusterfs.py