glusterfs: volume-<uuid>.info is not deleted when volume is deleted

Bug #1299182 reported by Thang Pham on 2014-03-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Low
Thang Pham

Bug Description

glusterfs creates a file named volume-<uuid>.info to hold volume data, mapping cinder snapshot UUIDs and filenames. When the volume and its associated snapshots are both deleted, this file remains. The expected behavior is when a volume (and all its snapshots) is deleted, the volume-<uuid>.info should be deleted as well.

To reproduce this bug:

1. Create a volume.
Verify the files under the mount point.
# ls <glusterfs_mount_point>
volume-4a5648f4-7687-46bc-bc50-3dcffe2624b4

2. Take volume snapshot.
Verify the files under the mount point.
# ls <glusterfs_mount_point>
volume-4a5648f4-7687-46bc-bc50-3dcffe2624b4
volume-4a5648f4-7687-46bc-bc50-3dcffe2624b4.37b7fcbf-9e60-4fef-8e82-861830f4cbce
volume-4a5648f4-7687-46bc-bc50-3dcffe2624b4.info

3. Delete the snapshot.
Verify the files under the mount point.
# ls <glusterfs_mount_point>
volume-4a5648f4-7687-46bc-bc50-3dcffe2624b4
volume-4a5648f4-7687-46bc-bc50-3dcffe2624b4.info

4. Delete the volume.
Verify the files under the mount point.
# ls <glusterfs_mount_point>
volume-4a5648f4-7687-46bc-bc50-3dcffe2624b4.info

* Note the volume-<uuid>.info continues to exists even though the volume and its snapshots are deleted.

Thang Pham (thang-pham) on 2014-03-28
Changed in cinder:
assignee: nobody → Thang Pham (thang-pham)
Thang Pham (thang-pham) on 2014-03-28
tags: added: glusterfs

Fix proposed to branch: master
Review: https://review.openstack.org/84222

Changed in cinder:
status: New → In Progress
Eric Harney (eharney) on 2014-04-01
Changed in cinder:
importance: Undecided → Low

Reviewed: https://review.openstack.org/84222
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=bf006ecb4a52db6300aa0fe1abe7df621e9f6174
Submitter: Jenkins
Branch: master

commit bf006ecb4a52db6300aa0fe1abe7df621e9f6174
Author: Thang Pham <email address hidden>
Date: Mon Mar 31 15:22:47 2014 -0400

    GlusterFS: Delete volume-<uuid>.info file when volume is deleted.

    glusterfs creates a file named volume-<uuid>.info to hold volume
    data, mapping cinder snapshot UUIDs and filenames. When the
    volume and its associated snapshots are both deleted, this file
    remains. This patch deletes the volume-<uuid>.info (if one exists)
    when the volume is deleted.

    Change-Id: Iefc8bd4768b30a75c648c4689d935daeab3afe98
    Closes-Bug: #1299182

Changed in cinder:
status: In Progress → Fix Committed
Changed in cinder:
milestone: none → icehouse-rc2

Reviewed: https://review.openstack.org/85370
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=d0262065921cb3015619bdce8ca1cae596cde9b2
Submitter: Jenkins
Branch: milestone-proposed

commit d0262065921cb3015619bdce8ca1cae596cde9b2
Author: Thang Pham <email address hidden>
Date: Mon Mar 31 15:22:47 2014 -0400

    GlusterFS: Delete volume-<uuid>.info file when volume is deleted.

    glusterfs creates a file named volume-<uuid>.info to hold volume
    data, mapping cinder snapshot UUIDs and filenames. When the
    volume and its associated snapshots are both deleted, this file
    remains. This patch deletes the volume-<uuid>.info (if one exists)
    when the volume is deleted.

    Change-Id: Iefc8bd4768b30a75c648c4689d935daeab3afe98
    Closes-Bug: #1299182
    (cherry picked from commit bf006ecb4a52db6300aa0fe1abe7df621e9f6174)

Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-04-17
Changed in cinder:
milestone: icehouse-rc2 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers