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

nautilus: rgw: multisite: radosgw-admin commands should not modify metadata on a non-master zone #29163

Merged
merged 2 commits into from Jul 30, 2019

Conversation

@smithfarm
Copy link
Contributor

commented Jul 22, 2019

Shilpa Jagannath added 2 commits Jul 3, 2019
rgw: Fail radosgw-admin commands on non-master zone that modify metadata
but with an option to override, allowing changes only on the local zone.

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

Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit 79d884e)
Added single check to avoid duplication. Included few more commands.
Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit 14c3b4b)

@smithfarm smithfarm self-assigned this Jul 22, 2019

@smithfarm smithfarm added this to the nautilus milestone Jul 22, 2019

@smithfarm smithfarm added bug fix core rgw and removed core labels Jul 22, 2019

@smithfarm smithfarm changed the title nautilus: multisite: radosgw-admin commands should not modify metadata on a non-master zone nautilus: rgw: multisite: radosgw-admin commands should not modify metadata on a non-master zone Jul 24, 2019

@smithfarm smithfarm requested a review from cbodley Jul 24, 2019

@yuriw

This comment has been minimized.

Copy link
Contributor

commented Jul 25, 2019

@yuriw yuriw merged commit a17b0a8 into ceph:nautilus Jul 30, 2019

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-40851-nautilus branch Aug 21, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.