Skip to content

ceph-volume: add call to ceph-bluestore-tool zap-device#59742

Merged
guits merged 1 commit intoceph:mainfrom
guits:wip-guits-cv-cbt-zap-device
Sep 25, 2024
Merged

ceph-volume: add call to ceph-bluestore-tool zap-device#59742
guits merged 1 commit intoceph:mainfrom
guits:wip-guits-cv-cbt-zap-device

Conversation

@guits
Copy link
Contributor

@guits guits commented Sep 11, 2024

BlueStore now writes its metadata at multiple offset on devices [1]. It means ceph-volume lvm zap doesn't remove BlueStore signature altogether. This can confuse ceph-volume when redeploying an OSD on a previously zapped device because there is still old BlueStore metadata on it.

ceph-volume should call ceph-bluestore-tool zap-device [2] in addition to the existing calls when wiping a device.

[1] #55374
[2] #59632

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

@guits guits force-pushed the wip-guits-cv-cbt-zap-device branch 2 times, most recently from 599a79b to a82c7dd Compare September 11, 2024 15:55
@guits
Copy link
Contributor Author

guits commented Sep 12, 2024

jenkins test api

@guits
Copy link
Contributor Author

guits commented Sep 12, 2024

needs #59762

@guits guits force-pushed the wip-guits-cv-cbt-zap-device branch 2 times, most recently from 41a8a4b to d3d0028 Compare September 13, 2024 08:37
BlueStore now writes its metadata at multiple offset on devices [1].
It means `ceph-volume lvm zap` doesn't remove BlueStore signature altogether.
This can confuse ceph-volume when redeploying an OSD on a previously
zapped device because there is still old BlueStore metadata on it.

ceph-volume should call `ceph-bluestore-tool zap-device` [2]
in addition to the existing calls when wiping a device.

[1] ceph#55374
[2] ceph#59632

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

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
@guits guits force-pushed the wip-guits-cv-cbt-zap-device branch from d3d0028 to dcf7439 Compare September 13, 2024 08:40
@guits
Copy link
Contributor Author

guits commented Sep 16, 2024

jenkins test make check

@guits
Copy link
Contributor Author

guits commented Sep 21, 2024

@guits
Copy link
Contributor Author

guits commented Sep 23, 2024

jenkins test windows

1 similar comment
@guits
Copy link
Contributor Author

guits commented Sep 23, 2024

jenkins test windows

@guits guits merged commit 06708cb into ceph:main Sep 25, 2024
@guits guits deleted the wip-guits-cv-cbt-zap-device branch September 25, 2024 06:43
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