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

jewel: rgw: S3 object versioning fails when applied on a non-master zone #11367

Merged
2 commits merged into from Oct 13, 2016

Conversation

mattbenjamin
Copy link
Contributor

@mattbenjamin mattbenjamin commented Oct 7, 2016

Was missing args, so it didn't sign requests with subresources
correctly when forwarding.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit dd1068c)
Fixes: http://tracker.ceph.com/issues/16494

Needed to keep input data around to be forwarded correctly. Also, master
does not send any data back, so don't try to parse anything.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 1eec00b)
@mattbenjamin mattbenjamin added this to the jewel milestone Oct 7, 2016
@ghost ghost changed the title jewel:rgw:fix for issue 16494 jewel: rgw: S3 object versioning fails when applied on a non-master zone Oct 10, 2016
@ghost ghost added bug-fix and removed needs-qa labels Oct 10, 2016
@ghost ghost self-assigned this Oct 10, 2016
ghost pushed a commit that referenced this pull request Oct 10, 2016
…n applied on a non-master zone

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 10, 2016
…n applied on a non-master zone

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 13, 2016
…n applied on a non-master zone

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Oct 13, 2016

@oritwas does this backport look good to merge ? It passed the rgw suite http://tracker.ceph.com/issues/17487#note-11.

@ghost ghost assigned oritwas Oct 13, 2016
@oritwas
Copy link
Member

oritwas commented Oct 13, 2016

lgtm

@ghost ghost merged commit abc8e3d into ceph:jewel Oct 13, 2016
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants