From 8f5e10d7ed8eead221cccb6a0ccf1afea310b0a9 Mon Sep 17 00:00:00 2001
From: Eric Harney <eharney@redhat.com>
Date: Fri, 30 Jun 2017 08:56:23 -0400
Subject: [PATCH] Don't use ignoreskipactivation for thin LVM

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

Closes-Bug: #1701545

Change-Id: I6b41e736f4c000205784fa97a1eadfccfe34722f
---
 cinder/brick/local_dev/lvm.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/cinder/brick/local_dev/lvm.py b/cinder/brick/local_dev/lvm.py
index 78db70078ed..83bf254e2ed 100644
--- a/cinder/brick/local_dev/lvm.py
+++ b/cinder/brick/local_dev/lvm.py
@@ -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.