openstack
/
cinder
Code Issues Proposed changes

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
This commit is contained in:
Eric Harney 2014-11-17 15:01:02 -05:00
parent 522c640bc3
commit 17221668fc
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -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.