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: s3tests-test-readwrite failing with 500 #11471

Merged
merged 1 commit into from Oct 20, 2016
Merged

jewel: rgw: s3tests-test-readwrite failing with 500 #11471

merged 1 commit into from Oct 20, 2016

Conversation

ghost
Copy link

@ghost ghost commented Oct 13, 2016

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

When concurrently setting acls on object/bucket, we could lose in a race.
Instead of retry, just return success (same effect as if we won and then
other writer overwrote us).

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 6e9a915)
@ghost ghost self-assigned this Oct 13, 2016
@ghost ghost added this to the jewel milestone Oct 13, 2016
@ghost ghost added bug-fix rgw labels Oct 13, 2016
ghost pushed a commit that referenced this pull request Oct 17, 2016
… 500

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 18, 2016
… 500

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost assigned cbodley Oct 20, 2016
@ghost
Copy link
Author

ghost commented Oct 20, 2016

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

@cbodley cbodley merged commit f6d7290 into ceph:jewel Oct 20, 2016
@theanalyst theanalyst changed the title jewel: s3tests-test-readwrite failing with 500 "jewel: rgw: s3tests-test-readwrite failing with 500" Nov 17, 2016
@theanalyst theanalyst changed the title "jewel: rgw: s3tests-test-readwrite failing with 500" jewel: rgw: s3tests-test-readwrite failing with 500 Nov 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants