Skip to content

ceph-volume: fix raw list for lvm devices#52429

Merged
guits merged 2 commits intoceph:mainfrom
guits:fix-raw-list
Jul 25, 2023
Merged

ceph-volume: fix raw list for lvm devices#52429
guits merged 2 commits intoceph:mainfrom
guits:fix-raw-list

Conversation

@guits
Copy link
Contributor

@guits guits commented Jul 13, 2023

raw list shouldn't list lvm OSDs.
This confuses the generic activate process given that it relies
on the raw list to detect which kind of OSD is being activated.

Fixes: https://tracker.ceph.com/issues/62002

Signed-off-by: Guillaume Abrioux gabrioux@ibm.com

Copy link
Contributor

@asm0deuz asm0deuz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@guits guits force-pushed the fix-raw-list branch 2 times, most recently from 09cbde7 to 1d79c33 Compare July 18, 2023 08:38
guits added 2 commits July 19, 2023 10:17
broken by bea9f4b
This commits fixes that regression.

Fixes: https://tracker.ceph.com/issues/62001

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
raw list shouldn't list lvm OSDs.
This confuses the generic activate process given that it relies
on the raw list to detect which kind of OSD is being activated.

Fixes: https://tracker.ceph.com/issues/62002

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
@guits
Copy link
Contributor Author

guits commented Jul 24, 2023

jenkins test make check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants