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

nautilus: qa/workunits/rbd: wait for nbd map to close after unmap #34736

Merged
merged 1 commit into from Apr 28, 2020

Conversation

smithfarm
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/44648


backport of #33898
parent tracker: https://tracker.ceph.com/issues/44567

this backport was staged using ceph-backport.sh version 15.1.1.389
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

The unmap action only sends a signal to the kernel to notify the
rbd-nbd daemon to disconnect. Therefore, it's possible that an
unmap followed by an immediate re-map to the same device might
fail since the unmap is still in-progress.

Fixes: https://tracker.ceph.com/issues/44567
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 80a3f18)
@smithfarm smithfarm added this to the nautilus milestone Apr 24, 2020
@smithfarm smithfarm requested a review from dillaman April 24, 2020 13:22
@smithfarm smithfarm added the rbd label Apr 24, 2020
Copy link

@dillaman dillaman left a comment

Choose a reason for hiding this comment

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

👍

@smithfarm
Copy link
Contributor Author

1 tests failed out of 168 Total Test time (real) = 969.99 sec
The following tests FAILED: 5 - run-tox-mgr-dashboard (Failed)

Unrelated failure. Ruled a PASS.

@yuriw
Copy link
Contributor

yuriw commented Apr 27, 2020

@yuriw
Copy link
Contributor

yuriw commented Apr 28, 2020

retest this please

@yuriw yuriw merged commit bfd919b into ceph:nautilus Apr 28, 2020
@smithfarm smithfarm deleted the wip-44648-nautilus branch May 5, 2020 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants