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

jewel: rbd: krbd-related CLI patches #11400

Merged
4 commits merged into from Oct 11, 2016
Merged

jewel: rbd: krbd-related CLI patches #11400

4 commits merged into from Oct 11, 2016

Conversation

idryomov
Copy link
Contributor

@idryomov idryomov commented Oct 10, 2016

@idryomov idryomov added this to the jewel milestone Oct 10, 2016
@idryomov idryomov added the rbd label Oct 10, 2016
@idryomov
Copy link
Contributor Author

Original master PRs: #11303 #11313 #11370.

@idryomov idryomov changed the title Backport krbd-related CLI patches jewel: backport krbd-related CLI patches Oct 10, 2016
@ghost ghost changed the title jewel: backport krbd-related CLI patches jewel: krbd-related CLI patches Oct 11, 2016
@ghost ghost self-assigned this Oct 11, 2016
@ghost ghost added the bug-fix label Oct 11, 2016
@ghost
Copy link

ghost commented Oct 11, 2016

@idryomov good to merge with a description of the krbd test that was run to verify it does work.

Reviewed-by: Loic Dachary <ldachary@redhat.com>

It wasn't clear from the man page that --size defaults to M.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit d38dc29)

Conflicts:
	doc/man/8/rbd.rst [ PR #9066 ("rbd: introduce a simple bench
	  for read") not in jewel ]
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit e857b78)
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit ba8f9ee)
Reuse rbd map -o infrastructure to expose rbd unmap options in
a similar fashion.  Currently it's just one bool option, but we may
need more in the future.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit 620f5e1)

Conflicts:
	doc/man/8/rbd.rst [ PR #9151 ("rbd: add methods to set and get
	  snapshot limits") not in jewel ]
@idryomov
Copy link
Contributor Author

Repushed with conflict descriptions.

@idryomov
Copy link
Contributor Author

These commits allow passing certain options to the rbd kernel module and don't bring in any new functionality. Tested manually by myself.

@ghost ghost merged commit 4769027 into jewel Oct 11, 2016
@idryomov idryomov deleted the wip-rbd-cli-jewel branch October 11, 2016 10:33
@theanalyst theanalyst changed the title jewel: krbd-related CLI patches "jewel: rbd: krbd-related CLI patches" Nov 17, 2016
@theanalyst theanalyst changed the title "jewel: rbd: krbd-related CLI patches" jewel: rbd: krbd-related CLI patches Nov 17, 2016
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant