Project

General

Profile

Actions

Backport #38200

closed

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

Added by Jan Fajerski about 5 years ago. Updated over 4 years ago.

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


Related issues 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Jan Fajerski about 5 years ago

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

Updated by Nathan Cutler about 5 years ago

  • Status changed from New to In Progress
Actions #3

Updated by Nathan Cutler about 5 years ago

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

Updated by Jan Fajerski over 4 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF