From: Eric Harney <eharney@redhat.com>
Date: Fri, 13 Dec 2013 16:18:56 +0000 (-0500)
Subject: LVM: Activate source snap LV when cloning from volume
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=673057320929d33095c998d477c2537567ccf74b;p=openstack-build%2Fcinder-build.git

LVM: Activate source snap LV when cloning from volume

Similar to the change made in:
01a2199 LVM: Activate source LV before cloning from it

Previous change only fixed this issue for clone-from-snapshot.

This activates the source snapshot LV when cloning from volume
as well, fixing the problem where cloning fails due to inability
to open the source LV device.

Change-Id: If48be8516fd71738cb83a91683c33d02f227b9d6
Related-Bug: 1252423
Closes-Bug: 1260786
---

diff --git a/cinder/tests/brick/fake_lvm.py b/cinder/tests/brick/fake_lvm.py
index a6f76ab9b..d6eec28f5 100644
--- a/cinder/tests/brick/fake_lvm.py
+++ b/cinder/tests/brick/fake_lvm.py
@@ -67,3 +67,6 @@ class FakeBrickLVM(object):
 
     def lv_has_snapshot(self, name):
         return False
+
+    def activate_lv(self, lv, is_snapshot=False):
+        pass
diff --git a/cinder/volume/drivers/lvm.py b/cinder/volume/drivers/lvm.py
index 3529fe955..220b865d1 100644
--- a/cinder/volume/drivers/lvm.py
+++ b/cinder/volume/drivers/lvm.py
@@ -309,6 +309,8 @@ class LVMVolumeDriver(driver.VolumeDriver):
                             self.configuration.lvm_type,
                             mirror_count)
 
+        self.vg.activate_lv(temp_snapshot['name'], is_snapshot=True)
+
         try:
             volutils.copy_volume(self.local_path(temp_snapshot),
                                  self.local_path(volume),