-
Notifications
You must be signed in to change notification settings - Fork 6k
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 radosgw-admin retcode #15257
Conversation
should check RGWBucketAdminOp::info retcode Signed-off-by: Shasha Lu <lu.shasha@eisoo.com>
src/rgw/rgw_admin.cc
Outdated
@@ -4491,7 +4491,8 @@ int main(int argc, const char **argv) | |||
case OPT_USER_RM: | |||
ret = user.remove(user_op, &err_msg); | |||
if (ret == -ENOENT) { | |||
cerr << err_msg << std::endl; | |||
cerr << "could not remove user: " << err_msg << std::endl; | |||
return -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.
can we just remove the special case for -ENOENT
here, and let if (ret < 0)
handle this below?
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.
thanks , already removed.
258b33f
to
0e43d2f
Compare
when user.remove got negative retcode should return Signed-off-by: Shasha Lu <lu.shasha@eisoo.com>
0e43d2f
to
6b57439
Compare
ping @cbodley |
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.
looks good, i'll include this in my next teuthology run
fix "radosgw-admin user rm" and "radosgw-admin bucket stats" retcode
Signed-off-by: Shasha Lu lu.shasha@eisoo.com