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

luminous: cls/user: cls_user_remove_bucket writes modified header #24855

Merged
merged 1 commit into from Nov 1, 2018

Conversation

Projects
None yet
3 participants
@cbodley
Contributor

cbodley commented Oct 31, 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)
@yuriw

This comment has been minimized.

Contributor

yuriw commented Oct 31, 2018

@cbodley cbodley requested a review from mattbenjamin Nov 1, 2018

@yuriw

yuriw approved these changes Nov 1, 2018

Reviewed-by: Casey Bodley cbodley@redhat.com

@yuriw yuriw merged commit 3149acf into ceph:luminous Nov 1, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment