From 17221668fcf5615fc0e9a805924f1176ee21ffcf Mon Sep 17 00:00:00 2001 From: Eric Harney Date: Mon, 17 Nov 2014 15:01:02 -0500 Subject: [PATCH] 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 --- cinder/brick/local_dev/lvm.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) 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. -- 2.45.2