]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix secure delete for thick LVM snapshots
authorrackerjoe <breu@breu.org>
Wed, 4 Sep 2013 20:31:42 +0000 (15:31 -0500)
committerJohn Griffith <john.griffith@solidfire.com>
Tue, 17 Sep 2013 23:16:48 +0000 (17:16 -0600)
commitf702fe7e30e4021895dac8e7ab243e5192f8182d
tree89db7f09677e4a583d03a78f2da8783c178aae86
parent7647ce8c4d3878002ae9898c7449a03fbf6c74b4
Fix secure delete for thick LVM snapshots

This change modifies the behaviour of the secure delete for thick
LVM snapshots to wipe the underlying COW of the snapshot LV
instead of the snapshot LV itself.

This change is necessary because the snapshot LV does not contain
exactly the same number of writable blocks as the original LV.  The
COW includes header information per COW block that identifies the
device as a COW device as well as the source and destination blocks
for the changed item.  The amount of metadata contained in the COW is
variable based on I/O performed on the snapshot.

This does not change the behavior of secure deletes on thin LVs
or secure deletes on the thick LV snapshot origin.

Closes-Bug: #1191812
Change-Id: I20e02b6c20d5ac539b5b5469e665fc986180f2e9
cinder/tests/api/contrib/test_admin_actions.py
cinder/tests/test_volume.py
cinder/volume/drivers/lvm.py