From e5f4168bf27ad1da5ebc22af85faa0e898ce2f0e Mon Sep 17 00:00:00 2001 From: wang yong Date: Thu, 6 Jul 2017 15:10:23 +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: I3e19c574aa76594898307682c4898d5459c2af76 --- cinder/brick/local_dev/lvm.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/cinder/brick/local_dev/lvm.py b/cinder/brick/local_dev/lvm.py index 78db70078ed..ed49a92e123 100644 --- a/cinder/brick/local_dev/lvm.py +++ b/cinder/brick/local_dev/lvm.py @@ -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: