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: multisite: fixes for zonegroup redirect #14319

Merged
merged 2 commits into from Apr 11, 2017

Conversation

Projects
None yet
2 participants
@cbodley
Contributor

cbodley commented Apr 4, 2017

cbodley added some commits Apr 4, 2017

rgw: use zonegroup's master zone endpoints for bucket redirect
if no zonegroup endpoints are set, fall back to master zone endpoints

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

Signed-off-by: Casey Bodley <cbodley@redhat.com>
rgw: fix for zonegroup redirect url
local dest_url variable was shadowing the one in the enclosing scope, so
the changes were not applied and no Location header was written on redirect

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

Signed-off-by: Casey Bodley <cbodley@redhat.com>
@yehudasa

lgtm

@cbodley

This comment has been minimized.

@cbodley cbodley merged commit 2dc5789 into ceph:master Apr 11, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@cbodley cbodley deleted the cbodley:wip-19488 branch Apr 11, 2017

dongbula pushed a commit to dongbula/ceph that referenced this pull request Jul 22, 2017

Merge pull request ceph#14319 from cbodley/wip-19488
rgw: multisite: fixes for zonegroup redirect

Reviewed-by: Yehuda Sadeh <yehuda@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment