diff options
author | Adam King <adking@redhat.com> | 2023-11-27 21:04:42 +0100 |
---|---|---|
committer | Adam King <adking@redhat.com> | 2023-11-29 19:57:00 +0100 |
commit | d3f1a0e1c0b98b9f1251837ecc8edc367e590dad (patch) | |
tree | 6b94ea8bf27e51401804e894440e968f1eb09c07 /qa/suites/fs/shell/distro | |
parent | Merge pull request #54578 from ronen-fr/wip-rf-dedup-clang (diff) | |
download | ceph-d3f1a0e1c0b98b9f1251837ecc8edc367e590dad.tar.xz ceph-d3f1a0e1c0b98b9f1251837ecc8edc367e590dad.zip |
python-common/drive_selection: fix limit with existing devices
When devices have already been used for OSDs, they are still
allowed to pass filtering as they are still needed for the
resulting ceph-volume lvm batch command. This was causing an
issue with limit however. Limit adds the devices we've found
that match the filter and existing OSD daemons tied to the spec.
This allows double counting of devices that hae been used for
OSDs, as they're counted in terms of being an existing device
and that they match the filter. To avoid this issue, devices
should only be counted towards the limit if they are not already
part of an OSD.
An additional note: The limit feature is only applied for
data devices, so there is no need to worry about the effect
of this change on selection of db, wal, or journal devices.
Also, we would still want to not count these devices if they
did end up passing the data device filter but had been used
for a db/wal/journal device previously.
Fixes: https://tracker.ceph.com/issues/63525
Signed-off-by: Adam King <adking@redhat.com>
Diffstat (limited to 'qa/suites/fs/shell/distro')
0 files changed, 0 insertions, 0 deletions