Project

General

Profile

Actions

Bug #58812

closed

ceph-volume prepare doesn't use partitions as-is anymore

Added by Nicolas Lindae about 1 year ago. Updated 4 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
% Done:

0%

Source:
Community (user)
Tags:
backport_processed
Backport:
pacific quincy reef
Regression:
Yes
Severity:
3 - minor
Reviewed:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The is_device function in ceph-volume/util/disk.py is supposed to return false for partitions. This is later used to not create LVMs in the prepare command, see https://docs.ceph.com/en/quincy/ceph-volume/lvm/prepare/#ceph-volume-lvm-prepare:

For both block.db and block.wal, partitions can be used as-is, and therefore are not made into logical volumes.

This behavior changed with this commit: https://github.com/ceph/ceph/commit/5705e10e809cdc9f70018263c54a63ac4a02809c
Without lsblk, the stats fallback is used for partitions, and that happens to not work. So the bug was kind of hidden until this commit.

It's easy to test, you can copy the is_device and _stat_is_device and test them locally, they return true for my partitions. ceph-volume prepare therefore makes logical volumes on top of the partitions.


Related issues 3 (0 open3 closed)

Copied to ceph-volume - Backport #63598: quincy: ceph-volume prepare doesn't use partitions as-is anymoreResolvedGuillaume AbriouxActions
Copied to ceph-volume - Backport #63599: reef: ceph-volume prepare doesn't use partitions as-is anymoreResolvedGuillaume AbriouxActions
Copied to ceph-volume - Backport #63677: pacific: ceph-volume prepare doesn't use partitions as-is anymoreResolvedKonstantin ShalyginActions
Actions

Also available in: Atom PDF