From: Eric Harney <eharney@redhat.com>
Date: Mon, 17 Nov 2014 20:01:02 +0000 (-0500)
Subject: Brick LVM: Remove self.lv_list
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=17221668fcf5615fc0e9a805924f1176ee21ffcf;p=openstack-build%2Fcinder-build.git

Brick LVM: Remove self.lv_list

This member isn't used anywhere, and now that requests for
specific LVs are overwriting self.lv_list with a subset of
the LVs that are present, it doesn't contain coherent info
anyway.

Change-Id: I4d661869012937cc40177e53c8546d5cd7c98cd9
---

diff --git a/cinder/brick/local_dev/lvm.py b/cinder/brick/local_dev/lvm.py
index 4c9264844..9d2cdf7ff 100644
--- a/cinder/brick/local_dev/lvm.py
+++ b/cinder/brick/local_dev/lvm.py
@@ -58,7 +58,6 @@ class LVM(executor.Executor):
         super(LVM, self).__init__(execute=executor, root_helper=root_helper)
         self.vg_name = vg_name
         self.pv_list = []
-        self.lv_list = []
         self.vg_size = 0.0
         self.vg_free_space = 0.0
         self.vg_lv_count = 0
@@ -287,10 +286,9 @@ class LVM(executor.Executor):
         :returns: List of Dictionaries with LV info
 
         """
-        self.lv_list = self.get_lv_info(self._root_helper,
-                                        self.vg_name,
-                                        lv_name)
-        return self.lv_list
+        return self.get_lv_info(self._root_helper,
+                                self.vg_name,
+                                lv_name)
 
     def get_volume(self, name):
         """Get reference object of volume specified by name.