From c1bd361b93b52a744ca01cec3e4adc6e5d6e25e4 Mon Sep 17 00:00:00 2001 From: wang yong Date: Fri, 7 Jul 2017 21:05:59 +0800 Subject: [PATCH] Remove the unnecessary pv_list assign during LVM object init 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 --- os_brick/local_dev/lvm.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/os_brick/local_dev/lvm.py b/os_brick/local_dev/lvm.py index f0aaca153..69c271575 100644 --- a/os_brick/local_dev/lvm.py +++ b/os_brick/local_dev/lvm.py @@ -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: