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

mimic: rgw: cls_user_remove_bucket does not write the modified cls_user_stats #24910

Merged
merged 1 commit into from Nov 20, 2018

Conversation

Projects
None yet
3 participants
@smithfarm
Copy link
Contributor

smithfarm commented Nov 3, 2018

cls/user: cls_user_remove_bucket writes modified header
if cls_user_remove_bucket() modifies the header with dec_header_stats(),
it should write the updated header back

Fixes: http://tracker.ceph.com/issues/36496

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit dac26de)

@smithfarm smithfarm added this to the mimic milestone Nov 3, 2018

@smithfarm smithfarm added the rgw label Nov 3, 2018

@smithfarm smithfarm requested review from mattbenjamin and cbodley Nov 3, 2018

@smithfarm smithfarm changed the title mimic: cls_user_remove_bucket does not write the modified cls_user_stats mimic: rgw: cls_user_remove_bucket does not write the modified cls_user_stats Nov 3, 2018

@cbodley

cbodley approved these changes Nov 5, 2018

@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Nov 19, 2018

@yuriw yuriw merged commit 9b43a43 into ceph:mimic Nov 20, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@smithfarm smithfarm deleted the smithfarm:wip-36533-mimic branch Nov 20, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment