Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

filestore-to-bluestore: umount partitions before zapping them #4871

Merged
merged 1 commit into from Dec 19, 2019

Conversation

guits
Copy link
Collaborator

@guits guits commented Dec 18, 2019

When an OSD is stopped, it leaves partitions mounted.
We must umount them before zapping them, otherwise error like "Device is
busy" will show up.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1729267

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

When an OSD is stopped, it leaves partitions mounted.
We must umount them before zapping them, otherwise error like "Device is
busy" will show up.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1729267

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
@guits guits changed the title [skip ci] filestore-to-bluestore: umount partitions before zapping them filestore-to-bluestore: umount partitions before zapping them Dec 18, 2019
@guits
Copy link
Collaborator Author

guits commented Dec 18, 2019

jenkins test centos-container-filestore_to_bluestore

@guits
Copy link
Collaborator Author

guits commented Dec 18, 2019

jenkins test centos-non_container-filestore_to_bluestore

2 similar comments
@guits
Copy link
Collaborator Author

guits commented Dec 18, 2019

jenkins test centos-non_container-filestore_to_bluestore

@guits
Copy link
Collaborator Author

guits commented Dec 19, 2019

jenkins test centos-non_container-filestore_to_bluestore

@guits
Copy link
Collaborator Author

guits commented Dec 19, 2019

unrelated failures

@guits guits merged commit 8056514 into master Dec 19, 2019
@guits guits deleted the guits-filestore2bluestore_dmcrypt branch December 19, 2019 08:22
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.

None yet

1 participant