openstack/cinder
Code Issues Proposed changes

Don't use ignoreskipactivation for thin LVM

Browse Source 
This causes failures for volume clone and
is not needed for thin.

Closes-Bug: #1701545

Change-Id: I6b41e736f4c000205784fa97a1eadfccfe34722f
This commit is contained in:
Eric Harney 2017-06-30 08:56:23 -04:00
parent 1e8d7fda4f
commit 8f5e10d7ed
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
cinder/brick/local_dev/lvm.py
View File

@ -672,7 +672,8 @@ class LVM(executor.Executor):
# and fails.
cmd = ['lvchange', '-a', 'y', '--yes']
if self.supports_lvchange_ignoreskipactivation:
if (self.supports_lvchange_ignoreskipactivation and
self.vg_thin_pool is None):
cmd.append('-K')
# If permanent=True is specified, drop the skipactivation flag in
# order to make this LV automatically activated after next reboot.