openstack/os-brick
Code Issues Proposed changes

Remove the unnecessary pv_list assign during LVM object init

Browse Source 
The pv_list of LVM object actually wanted to be a list of dict of
physical volumes' info, but in the routine of creating a new vg,
we assign a list of physical volumes' name to it, which is unnecessary,
and also confuse people.

Change-Id: I5f74200e6df954258f261a30e9f5a5b56c6def2f
This commit is contained in:
wang yong 2017-07-07 21:05:59 +08:00 committed by Sean McGinnis
parent effd16e33e
commit c1bd361b93
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
os_brick/local_dev/lvm.py
View File

@ -82,8 +82,6 @@ class LVM(executor.Executor):
'LVM_SYSTEM_DIR=' + lvm_sys_dir]
if create_vg and physical_volumes is not None:
self.pv_list = physical_volumes
try:
self._create_vg(physical_volumes)
except putils.ProcessExecutionError as err: