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: bucket resharding should not update bucket ACL or user stats #20038

Merged
merged 1 commit into from Feb 5, 2018

Conversation

oritwas
Copy link
Member

@oritwas oritwas commented Jan 21, 2018

bucket link op resets bucket ACL attr using rgw_link_bucket instead

Fixes: http://tracker.ceph.com/issues/22742
Fixes: http://tracker.ceph.com/issues/22124

bucket link op resets bucket ACL using rgw_link_bucket instead

Fixes: http://tracker.ceph.com/issues/22124
Fixes: http://tracker.ceph.com/issues/22742
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
@oritwas oritwas changed the title rgw: bucket resharding should rewrite bucket attrs after link rgw: bucket resharding should not update bucket ACL or user stats Feb 5, 2018
@theanalyst
Copy link
Member

Is there anyway for resetting stats of already resharded buckets with the old version after upgrading? Does resharding again on an already resharded bucket help?

@yuriw
Copy link
Contributor

yuriw commented Feb 5, 2018

@oritwas
seems to have conflict

  • branch wip-rgw-22742 -> FETCH_HEAD
    warning: Failed to merge submodule src/rocksdb (not checked out)
    CONFLICT (rename/add): Rename src/test/ubuntu-12.04/Dockerfile.in->src/test/rbd-ggate.sh in 1223baf. src/test/rbd-ggate.sh added in HEAD

@oritwas
Copy link
Member Author

oritwas commented Feb 5, 2018

@yuriw, this doesn't change rocksdb submodule, very strange

@oritwas oritwas merged commit 19b430f into ceph:master Feb 5, 2018
@theanalyst
Copy link
Member

@yuriw the conflict was because the pr is for master, we need to raise another one for luminous for the batch to build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants