openstack/cinder
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: I3e19c574aa76594898307682c4898d5459c2af76
This commit is contained in:
wang yong 2017-07-06 15:10:23 +08:00
parent 30431abf2a
commit e5f4168bf2
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -91,8 +91,6 @@ class LVM(executor.Executor):
LVM.LVM_CMD_PREFIX = _lvm_cmd_prefix
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: