-
Notifications
You must be signed in to change notification settings - Fork 251
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
cephfs/admin: Add API to fetch volume info #846
Conversation
fe07036
to
487a8b2
Compare
It would be good to add a line to the desciption for dpulls to key off of since this is based on the work in 844. I think |
487a8b2
to
7b31598
Compare
🎉 All dependencies have been resolved ! |
This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes? 🙏 |
7b31598
to
05b94c5
Compare
@Mergifyio rebase |
Signed-off-by: Anoop C S <anoopcs@cryptolab.net>
Signed-off-by: Anoop C S <anoopcs@cryptolab.net>
Signed-off-by: Anoop C S <anoopcs@cryptolab.net>
✅ Branch has been successfully rebased |
05b94c5
to
cac917d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good. In and ideal world I would also have a few unit tests that exercise just the parseVolumeInfo function, but since the structs in question are pretty simple I won't require it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Checklist
//go:build ceph_preview
make api-update
to record new APIsFixes #695