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: fix the problem of rgw website configure 'RedirectAllRequestsTo' failed to sync to slave zone #15036

Merged
merged 1 commit into from Jun 16, 2017

Conversation

Projects
None yet
3 participants
@joke-lee
Contributor

joke-lee commented May 10, 2017

hi, when i enable bucket website and put

<WebsiteConfiguration><RedirectAllRequestsTo><HostName>google.com</HostName></RedirectAllRequestsTo></WebsiteConfiguration>

to the bucket (the request send to master zonegroup master zone),but i cannot get the website configue from the master zonegroup slave zone. (send GET /ms-bucket1/?website request to slave zone).it return empty xml just like

<?xml version="1.0" encoding="UTF-8"?><WebsiteConfiguration xmlns="http://s3.amazonaws.com/doc/2006-03-01/"></WebsiteConfiguration>

we need to sync the redirect_all too

rgw-multisite: fix the problem of rgw website configure 'RedirectAllR…
…equestsTo' failed to sync to slave zone

Signed-off-by: yuliyang <yuliyang@cmss.chinamobile.com>
@joke-lee

This comment has been minimized.

Contributor

joke-lee commented May 31, 2017

@tchaikov How about progress about this review ?

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented May 31, 2017

@joke-lee you might need to ping rgw developers for receiving reviews.

@cbodley cbodley self-assigned this May 31, 2017

@joke-lee

This comment has been minimized.

Contributor

joke-lee commented May 31, 2017

@cbodley hi, can you help to review this?

@joke-lee

This comment has been minimized.

Contributor

joke-lee commented May 31, 2017

@yehudasa hi, can you help to review this?

@joke-lee

This comment has been minimized.

Contributor

joke-lee commented May 31, 2017

@liewegas hi, can you help to review this?

@cbodley

This comment has been minimized.

Contributor

cbodley commented May 31, 2017

hi @joke-lee, this looks good but i'd like to see it tested as part of #15082

@cbodley cbodley added the needs-test label May 31, 2017

@cbodley cbodley removed the needs-test label Jun 14, 2017

@joke-lee

This comment has been minimized.

Contributor

joke-lee commented Jun 15, 2017

test this please

@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 16, 2017

jenkins test this please

1 similar comment
@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 16, 2017

jenkins test this please

@cbodley cbodley merged commit d6808ae into ceph:master Jun 16, 2017

6 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
Unmodified Submodules submodules for project are unmodified
Details
arm64 make check arm64 make check succeeded
Details
default Build finished.
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