]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Add deactivate step to extend_lv
authorCurt Bruns <curt.e.bruns@intel.com>
Mon, 6 Jul 2015 15:09:41 +0000 (15:09 +0000)
committerCurt Bruns <curt.e.bruns@intel.com>
Tue, 28 Jul 2015 18:46:25 +0000 (11:46 -0700)
commit314611f79afa6fe79f3d5022c814772048669bb0
treedbc4fb306f81b8a44c518fecc31df4a1bcc4467f
parenta5c988f93eda5581899ad67c514ed1cb5666ac27
Add deactivate step to extend_lv

Extending a linear LVM volume that has a snapshot requires
that the LV be deactivated explicitly prior to extending.

This adds a deactivate_lv method and calls it prior to
issuing the extend call. If auto_activation_volume_list
is not defined lvm.conf, then volumes will be reactivated
automatically after the extend.  If auto_activation_volume_list
is defined, then volumes that should be automatically
reactivated should be added to the auto_activation_volume_list
or they won't be activated automatically.
NOTE: This doesn't apply to thin provisioned LVs as they don't
require the deactivation step.

DocImpact: When a user extends LVM Volumes with a snapshot, the
volumes will be deactivated.  Their re-activation is automatic,
unless auto_activation_volume_list is defined in lvm.conf.
See lvm.conf for more info.  Thin provisioned LVM Volumes
will not be deactivated as they don't require it.

Co-Authored-By: John Griffith <john.griffith8@gmail.com>
Change-Id: If746625cfe658c3528525dc7f4bf869f1c9704dc
Closes-Bug: #1470558
cinder/brick/local_dev/lvm.py
cinder/tests/unit/brick/test_brick_lvm.py
cinder/tests/unit/test_srb.py