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-nbd: support disabling auto-exclusive lock transition logic #11438

Merged
merged 1 commit into from Oct 19, 2016

Conversation

Projects
None yet
2 participants
@trociny
Contributor

trociny commented Oct 12, 2016

Fixes: http://tracker.ceph.com/issues/17488
Signed-off-by: Mykola Golub mgolub@mirantis.com

Show outdated Hide outdated src/tools/rbd_nbd/rbd-nbd.cc
@@ -64,13 +64,15 @@ static void usage()
<< " --device <device path> Specify nbd device path\n"
<< " --read-only Map readonly\n"
<< " --nbds_max <limit> Override for module param\n"
<< " --single-client Forbid other clients write\n"

This comment has been minimized.

@dillaman

dillaman Oct 12, 2016

Contributor

Minor: how about just "--exclusive" or "--exclusive-client"?

@dillaman

dillaman Oct 12, 2016

Contributor

Minor: how about just "--exclusive" or "--exclusive-client"?

@dillaman

This comment has been minimized.

Show comment
Hide comment
@dillaman

dillaman Oct 12, 2016

Contributor

... also need to update the rbd CLI help.t cram test case

Contributor

dillaman commented Oct 12, 2016

... also need to update the rbd CLI help.t cram test case

Mykola Golub
@trociny

This comment has been minimized.

Show comment
Hide comment
@trociny

trociny Oct 12, 2016

Contributor

@dillaman Thanks, updated.

Contributor

trociny commented Oct 12, 2016

@dillaman Thanks, updated.

@dillaman

This comment has been minimized.

Show comment
Hide comment
@dillaman

dillaman Oct 12, 2016

Contributor

lgtm

Contributor

dillaman commented Oct 12, 2016

lgtm

@dillaman dillaman self-assigned this Oct 17, 2016

@dillaman dillaman merged commit 8743184 into ceph:master Oct 19, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment