From: Guillaume Abrioux <gabrioux@redhat.com>
Date: Tue, 21 Jun 2022 07:53:43 +0000 (+0200)
Subject: ceph-volume: rename _get_pv_paths() in util/device.py
X-Git-Tag: v16.2.11~431^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=54780e80fde7dc7142c01f855770b138cde65a2b;p=ceph.git

ceph-volume: rename _get_pv_paths() in util/device.py

This name is misleading, let's rename it for something clearer.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 325b6f50e88d8ee046fdba0b2bdb17a11577a2f6)
---

diff --git a/src/ceph-volume/ceph_volume/util/device.py b/src/ceph-volume/ceph_volume/util/device.py
index 1479de858d66..0753345a06be 100644
--- a/src/ceph-volume/ceph_volume/util/device.py
+++ b/src/ceph-volume/ceph_volume/util/device.py
@@ -279,7 +279,7 @@ class Device(object):
             # VGs, should we consider it as part of LVM? We choose not to
             # here, because most likely, we need to use VGs from this PV.
             self._is_lvm_member = False
-            for path in self._get_pv_paths():
+            for path in self._get_device_with_partitions_list():
                 vgs = lvm.get_device_vgs(path)
                 if vgs:
                     self.vgs.extend(vgs)
@@ -292,7 +292,7 @@ class Device(object):
                     self.lvs.extend(lvm.get_device_lvs(path))
         return self._is_lvm_member
 
-    def _get_pv_paths(self):
+    def _get_device_with_partitions_list(self):
         """
         For block devices LVM can reside on the raw block device or on a
         partition. Return a list of paths to be checked for a pv.