]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph-volume: create LVs when using partitions 58219/head
authorGuillaume Abrioux <gabrioux@ibm.com>
Mon, 15 Apr 2024 08:28:24 +0000 (08:28 +0000)
committerGuillaume Abrioux <gabrioux@redhat.com>
Mon, 24 Jun 2024 07:25:31 +0000 (07:25 +0000)
commit [1] introduced a behavior change.
`ceph-volume lvm prepare` used to create VGs/LVs when it was passed partitions
for db and/or wal devices. Since commit [1] has been introduced, it made ceph-volume
consume the partition directly, it doesn't create LV anymore. Although this
doesn't prevent from creating OSDs, this is a behavior change.

Fixes: https://tracker.ceph.com/issues/65477
[1] 1e7223281fa044c9653633e305c0b344e4c9b3a4

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 11f0173f1299ed6d137159dcf62c4a7a9fd20f82)

src/ceph-volume/ceph_volume/objectstore/lvmbluestore.py

index 5dc46361e92ec71ddfb549cba0882facacfcafcb..0fc3c68f8c6d856f88f36d6c1be643a1be4e90fe 100644 (file)
@@ -249,8 +249,8 @@ class LvmBlueStore(BlueStore):
             tags['ceph.%s_uuid' % device_type] = lv_uuid
             tags['ceph.%s_device' % device_type] = path
             lv.set_tags(tags)
-        elif disk.is_device(device_name):
-            # We got a disk, create an lv
+        elif disk.is_partition(device_name) or disk.is_device(device_name):
+            # We got a disk or a partition, create an lv
             lv_type = "osd-{}".format(device_type)
             name_uuid = system.generate_uuid()
             kwargs = {