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

rbd: fix default pool handling for krbd map/unmap #36854

Merged
merged 2 commits into from Aug 28, 2020

Conversation

idryomov
Copy link
Contributor

Broken with #36380.

The default pool name does not get passed to the kernel since commit
96f05a7 ("rbd: delay determination of default pool name").  The
kernel ends up interpreting the image name as the pool name (and the
snapshot name as the image name).

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
It is no longer a global variable, so rename the ones that have to do
with options for unmapping (currently just "force").

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
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.

lgtm

@idryomov
Copy link
Contributor Author

@idryomov
Copy link
Contributor Author

The run isn't clean due to a kernel issue and, now that we connect to the cluster on "rbd map", an assert is triggered in the messenger on qa/workunits/rbd/huge-tickets.sh. I'll address that separately.

@idryomov idryomov merged commit 46c9129 into ceph:master Aug 28, 2020
@idryomov idryomov deleted the wip-krbd-default-pool-fix branch August 28, 2020 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants