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

Bug #1299182 reported by Thang Pham
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
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.

Tags: glusterfs
Thang Pham (thang-pham)
Changed in cinder:
assignee: nobody → Thang Pham (thang-pham)
Thang Pham (thang-pham)
tags: added: glusterfs
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: New → In Progress
Eric Harney (eharney)
Changed in cinder:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (milestone-proposed)

Fix proposed to branch: milestone-proposed
Review: https://review.openstack.org/85370

Changed in cinder:
milestone: none → icehouse-rc2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (milestone-proposed)

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)
Changed in cinder:
milestone: icehouse-rc2 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.