GPFS Driver does not limit clone depth for snapshots

Bug #1227366 reported by Bill Owen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Bill Owen

Bug Description

GPFS Driver does not limit clone depth for snapshots - there is a potential for snapshot clones to grow without bound regardless of the setting of gpfs_max_clone_depth config flag.

Tags: drivers gpfs
Mike Perez (thingee)
tags: added: drivers gpfs
Bill Owen (billowen)
Changed in cinder:
assignee: nobody → Bill Owen (billowen)
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/47264

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

Reviewed: https://review.openstack.org/47264
Committed: http://github.com/openstack/cinder/commit/cbb057169094e9d633162cc5645fa19a82325d49
Submitter: Jenkins
Branch: master

commit cbb057169094e9d633162cc5645fa19a82325d49
Author: Bill Owen <email address hidden>
Date: Wed Sep 18 14:30:07 2013 -0700

    GPFS Driver missing clone depth limit for snapshots

    GPFS driver is fixed to handle snapshot clones correctly. Previously,
    these were allowed to grow without respecting limit defined in config
    flag gpfs_max_clone_depth. This change adds the depth check operation
    in create_snapshot.

    To ensure that all clone files are cleaned up, the delete_snapshot
    method now marks snapshots to be deleted with ts file extension
    and attempts to delete the snapshot. If the snapshot cannot be
    deleted because it has clone children, it will be deleted when the
    child is deleted.

    Closes-Bug: #1227366

    Change-Id: I4fb2a720b55dbe033159e6fb341f6e2f1508776e

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-rc1 → 2013.2
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.