Actions
Backport #38200
closedmimic: Change in util/disk.py get_devices() breaks inventory
Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
-
Release:
mimic
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
This commit https://github.com/ceph/ceph/commit/7f3c3598491e556ec1b93ea0a54f52144202fab6
breaks the inventory functionality as the result of get_devices no contains partitions of a device and these partition entries are not fully populated. E.g.:
{'/dev/vda': {'human_readable_size': '40.00 GB', 'locked': 1, 'model': '', 'nr_requests': '256', 'partitions': {'vda1': {'holders': [], 'sectors': '83884032', 'sectorsize': 512, 'size': '40.00 GB', 'start': '2048'}}, 'path': '/dev/vda', 'removable': '0', 'rev': '', 'ro': '0', 'rotational': '1', 'sas_address': '', 'sas_device_handle': '', 'scheduler_mode': 'mq-deadline', 'sectors': 0, 'sectorsize': '512', 'size': 42949672960.0, 'support_discard': '', 'vendor': '0x1af4'}, '/dev/vda1': {'holders': [], 'sectors': '83884032', 'sectorsize': 512, 'size': '40.00 GB', 'start': '2048'}, }
Why are partitions considered top-level devices now?
Backport: https://github.com/ceph/ceph/pull/26294
Actions