summaryrefslogtreecommitdiffstats
path: root/qa/suites/fs/shell/distro
diff options
context:
space:
mode:
authorAdam King <adking@redhat.com>2023-11-27 21:04:42 +0100
committerAdam King <adking@redhat.com>2023-11-29 19:57:00 +0100
commitd3f1a0e1c0b98b9f1251837ecc8edc367e590dad (patch)
tree6b94ea8bf27e51401804e894440e968f1eb09c07 /qa/suites/fs/shell/distro
parentMerge pull request #54578 from ronen-fr/wip-rf-dedup-clang (diff)
downloadceph-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