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

rgw:modify command stucking when operating radosgw-admin metadata list user #7032

Merged
merged 1 commit into from Jan 6, 2016

Conversation

liupeiyang
Copy link
Contributor

when no user exists,the command stucks by operating radosgw-admin metadata list user .It seems all right now

Signed-off-by: Peiyang Liu liu.peiyang@h3c.com

@liupeiyang
Copy link
Contributor Author

@yehudasa Could you take some time to look at it? commit #7032

@liupeiyang
Copy link
Contributor Author

@oritwas Could you look at it? #7032

if (ret == -ENOENT) {
if (truncated)
*truncated = false;
return 0;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't we return here -ENOENT?

@liupeiyang
Copy link
Contributor Author

@oritwas it should be better. Is that so?

 
 if (ret < 0 && ret != -ENOENT)
        return ret;         
 if (ret == -ENOENT) {
       if (truncated)
          *truncated = false;
       return -ENOENT;

@liupeiyang liupeiyang force-pushed the wip_metadata_list branch 2 times, most recently from 0f9e486 to 2766d29 Compare January 6, 2016 03:25
@liupeiyang
Copy link
Contributor Author

I have revised the commit @oritwas

…t user

When no user exists,the command stucks by operating radosgw-admin metadata list user. It seems all right now.

Signed-off-by: Peiyang Liu <liu.peiyang@h3c.com>
oritwas added a commit that referenced this pull request Jan 6, 2016
rgw:modify command stucking when operating radosgw-admin metadata list user
@oritwas oritwas merged commit 9cf7cbd into ceph:master Jan 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants