-
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
rgw/admin: Add missing uid member to usage struct #848
Conversation
2c0b99f
to
e382eaf
Compare
@thotz can you PTAL? |
Thanks for the review @thotz . Are we good to merge the PR in that case? |
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.
@Mergifyio rebase |
❌ Unable to rebase: user
|
Ok. I'll try once again. |
@Mergifyio rebase |
❌ Unable to rebase: user
|
Authorized..let's see now. |
@Mergifyio rebase |
**Description:** Added the uid member to the usage type. We don't need to handle the uid here, as the ceph funtion handles it in the backend. Fixes: ceph#758 Signed-off-by: Nikhil-Ladha <nikhilladha1999@gmail.com>
✅ Branch has been successfully rebased |
e382eaf
to
63247a5
Compare
Description
Added the uid member to the usage type.
We don't need to handle the uid here, as the ceph function handles it in the backend.
Fixes: #758
Signed-off-by: Nikhil-Ladha nikhilladha1999@gmail.com
Checklist
//go:build ceph_preview
make api-update
to record new APIs