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: Improve error reporting from rbd feature enable/disable #13157

Merged
merged 1 commit into from Jan 31, 2017

Conversation

smithfarm
Copy link
Contributor

Currently while enabling or disabling rbd feature command does not
give missing dependency for eg: attempting to enable the journaling
feature on an image that doesn't have the exclusive-lock feature
enabled should give missing dependency error message.

Fixes: http://tracker.ceph.com/issues/16985

Reported-by:  Jason Dillaman <dillaman@redhat.com>
Signed-off-by: Gaurav Kumar Garg <garg.gaurav52@gmail.com>
(manual cherry pick of bd023cf)
@smithfarm
Copy link
Contributor Author

@dillaman Passed an rbd suite at http://tracker.ceph.com/issues/17851#note-51 - ok to merge?

@smithfarm smithfarm changed the title jewel: Improve error reporting from rbd feature enable/disable jewel: rbd: Improve error reporting from rbd feature enable/disable Jan 31, 2017
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 smithfarm merged commit 26798a1 into ceph:jewel Jan 31, 2017
@smithfarm smithfarm deleted the wip-18434-jewel branch January 31, 2017 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants