Actions
Bug #24971
closedsystem wide validation of devices
Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
-
% Done:
0%
Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
Implement a system/layout for validating devices. This is tricky because some devices might be fine if they belong to LVM already, while
others shouldn't.
This is needed for the `lvm batch` commands, but also other sub-commands would benefit right away.
Additional validation/filter functions should be added, a few are already implemented in the choose_disk module in ceph-ansible, like a read-only device.
Updated by Alfredo Deza almost 5 years ago
- Status changed from New to Resolved
This was completed with `util/device.py` (https://github.com/ceph/ceph/blob/master/src/ceph-volume/ceph_volume/util/device.py) and its usage in all commands via argument validators: https://github.com/ceph/ceph/blob/master/src/ceph-volume/ceph_volume/util/arg_validators.py#L9
Actions