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: rgw: resharding produces invalid values of bucket stats #24527

Merged
merged 1 commit into from Oct 15, 2018

Conversation

Projects
None yet
3 participants
@theanalyst
Member

theanalyst commented Oct 10, 2018

rgw: copy actual stats from the source shards during reshard
Currently we don't copy the actual_stats field during reshard, which makes
resharded buckets show a size_utilized as 0, which further has the problem that
a subsequent object removal would subtract the object size from the 0 size
utilized showing up large uint64_t values. Copy the size_actual from the source
object in both cls and in reshard_process. This will fix the new buckets,
existing buckets will still have to go through a bucket check --fix for their
stats to be corrected.

Fixes: http://tracker.ceph.com/issues/36290
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
(cherry picked from commit beb9063)

@theanalyst theanalyst changed the title from 10 to luminous: resharding produces invalid values of bucket stats Oct 10, 2018

@theanalyst theanalyst added this to the luminous milestone Oct 10, 2018

@theanalyst theanalyst requested a review from cbodley Oct 10, 2018

@smithfarm smithfarm changed the title from luminous: resharding produces invalid values of bucket stats to luminous: rgw: resharding produces invalid values of bucket stats Oct 10, 2018

@smithfarm smithfarm added the needs-qa label Oct 10, 2018

@theanalyst theanalyst merged commit e35254c into ceph:luminous Oct 15, 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