Project

General

Profile

Backport #38201

luminous: Change in util/disk.py get_devices() breaks inventory

Added by Jan Fajerski almost 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
-
Release:
luminous
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?

https://github.com/ceph/ceph/pull/26295


Related issues

Copied from ceph-volume - Bug #37506: Change in util/disk.py get_devices() breaks inventory Resolved 12/03/2018

History

#1 Updated by Jan Fajerski almost 4 years ago

  • Copied from Bug #37506: Change in util/disk.py get_devices() breaks inventory added

#2 Updated by Nathan Cutler almost 4 years ago

  • Status changed from Pending Backport to New

#3 Updated by Nathan Cutler almost 4 years ago

  • Status changed from New to In Progress

#4 Updated by Nathan Cutler almost 4 years ago

  • Subject changed from [luminous] Change in util/disk.py get_devices() breaks inventory to luminous: Change in util/disk.py get_devices() breaks inventory

#5 Updated by Jan Fajerski over 3 years ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF