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: fix wrong debug related to user ACLs in rgw_build_bucket_policies() #14369
Conversation
src/rgw/rgw_op.cc
Outdated
@@ -492,7 +492,7 @@ int rgw_build_bucket_policies(RGWRados* store, struct req_state* s) | |||
map<string, bufferlist> uattrs; | |||
|
|||
ret = rgw_get_user_attrs_by_uid(store, acct_acl_user.uid, uattrs); | |||
if (!ret) { | |||
if (! ret) { |
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.
re: CodingStyle, the google style guide says:
// No spaces separating unary operators and their arguments.
x = -5;
++x;
if (x && !y)
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.
Sure, fixing right now.
e44c61a
to
447bd72
Compare
jenkins test this please |
@rzarzynski rebase please |
@rzarzynski can you please rebase? the conflict looks relevent to this change in error handling |
@rzarzynski pls fix/rebase --- pr 14369 --- pulling https://github.com/rzarzynski/ceph.git branch wip-rgw-19514
|
…s(). Fixes: http://tracker.ceph.com/issues/19514 Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
447bd72
to
21c76f9
Compare
jenkins retest this please (docs failure because of network issues) |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Fixes: http://tracker.ceph.com/issues/19514
Signed-off-by: Radoslaw Zarzynski rzarzynski@mirantis.com