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 3 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 #1

Updated by Guillaume Abrioux 11 months ago

  • Status changed from New to In Progress
  • Assignee set to Teoman ONAY
Actions #2

Updated by Teoman ONAY 7 months ago

  • Pull request ID set to 53798
Actions #3

Updated by Guillaume Abrioux 5 months ago

  • Status changed from In Progress to Fix Under Review
Actions #4

Updated by Guillaume Abrioux 5 months ago

  • Backport set to reef,quincy
Actions #5

Updated by Guillaume Abrioux 5 months ago

  • Status changed from Fix Under Review to Pending Backport
Actions #6

Updated by Backport Bot 5 months ago

  • Copied to Backport #63598: quincy: ceph-volume prepare doesn't use partitions as-is anymore added
Actions #7

Updated by Backport Bot 5 months ago

  • Copied to Backport #63599: reef: ceph-volume prepare doesn't use partitions as-is anymore added
Actions #8

Updated by Backport Bot 5 months ago

  • Tags set to backport_processed
Actions #9

Updated by Konstantin Shalygin 5 months ago

  • Source set to Community (user)
  • Backport changed from reef,quincy to reef,quincy,quincy
Actions #10

Updated by Konstantin Shalygin 5 months ago

  • Target version set to v19.0.0
  • Tags deleted (backport_processed)
  • Affected Versions v16.2.14, v17.2.7, v18.2.1 added
  • Affected Versions deleted (v17.2.5)
Actions #11

Updated by Konstantin Shalygin 5 months ago

  • Backport changed from reef,quincy,quincy to pacific quincy reef
Actions #12

Updated by Backport Bot 5 months ago

  • Copied to Backport #63677: pacific: ceph-volume prepare doesn't use partitions as-is anymore added
Actions #13

Updated by Backport Bot 5 months ago

  • Tags set to backport_processed
Actions #14

Updated by Guillaume Abrioux 3 months ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF