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

luminous: ceph_volume_client: perform snapshot operations in #17921

Merged
merged 1 commit into from Oct 29, 2017

Conversation

batrick
Copy link
Member

@batrick batrick commented Sep 22, 2017

... client configured snap directory name, instead of in hard-coded
'.snap' directory.

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

Signed-off-by: Ramana Raja <rraja@redhat.com>
(cherry picked from commit f4fc172)
@batrick batrick added cephfs Ceph File System wip-yuri-testing labels Sep 22, 2017
@batrick batrick modified the milestone: luminous Sep 25, 2017
@batrick
Copy link
Member Author

batrick commented Sep 25, 2017

Delaying until after 12.2.1.

@theanalyst
Copy link
Member

This passed a teuthology run at http://tracker.ceph.com/issues/21830#FS and will be merged when approved

@ajarr
Copy link
Contributor

ajarr commented Oct 29, 2017

@batrick, I'm unable to approve this change. It's a simple fix, and has been in review for a while. Can you please move this along?

@batrick batrick removed the request for review from jcsp October 29, 2017 17:52
@batrick
Copy link
Member Author

batrick commented Oct 29, 2017

Ramana, you can also just a comment and my script will pick up your review (but I realize that won't help with the "Review Required" github barrier)

Reviewed-by: Ramana ...

@batrick batrick merged commit d5e5834 into ceph:luminous Oct 29, 2017
batrick added a commit that referenced this pull request Oct 29, 2017
* refs/pull/17921/head:
	ceph_volume_client: perform snapshot operations in

Reviewed-by: Amit Kumar <amitkuma@redhat.com>
Reviewed-by: Ramana Raja <rraja@redhat.com>
@batrick batrick deleted the i21514 branch October 29, 2017 17:57
@batrick batrick restored the i21514 branch May 23, 2018 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cephfs Ceph File System
Projects
None yet
6 participants