Project

General

Profile

Bug #12286

rgw: radosgw-admin dumps user info twice

Added by Yehuda Sadeh about 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

when doing 'subuser modify'


Related issues

Copied to rgw - Backport #12400: rgw: radosgw-admin dumps user info twice Resolved 07/10/2015

Associated revisions

Revision c604dd97 (diff)
Added by gu ce about 8 years ago

Fixes: #12286 radosgw-admin: after subuser modify print only once user info.

remove rgw_admin.cc OPT_SUBUSER_MODIFY, show_user_info code block.

switch (opt_cmd) {
...
case OPT_SUBUSER_MODIFY:
show_user_info(info, formatter); //show first time (remove this)
break;
...
}

// output the result of a user operation
if (output_user_info) {
...
show_user_info(info, formatter); //show second time
}

test fix:
before: after subuser modify print twice user info.
after changes, do the same procedure, print only once user info.

Signed-off-by: guce

Revision e4b55b39 (diff)
Added by gu ce about 8 years ago

Fixes: #12286 radosgw-admin: after subuser modify print only once user info.

remove rgw_admin.cc OPT_SUBUSER_MODIFY, show_user_info code block.

switch (opt_cmd) {
...
case OPT_SUBUSER_MODIFY:
show_user_info(info, formatter); //show first time (remove this)
break;
...
}

// output the result of a user operation
if (output_user_info) {
...
show_user_info(info, formatter); //show second time
}

test fix:
before: after subuser modify print twice user info.
after changes, do the same procedure, print only once user info.

Signed-off-by: guce
(cherry picked from commit c604dd97fc179e5c2f640818c0f6e7cf99701947)

History

#1 Updated by Yehuda Sadeh about 8 years ago

  • Status changed from Fix Under Review to Pending Backport

#2 Updated by Nathan Cutler about 8 years ago

  • Assignee set to Yehuda Sadeh

#3 Updated by Nathan Cutler about 8 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF