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.

Sign up for GitHub

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

Jump to bottom

jewel: rgw: multisite: ECANCELED & 500 error on bucket delete #12044

Merged
1 commit merged into from Feb 15, 2017
Merged

jewel: rgw: multisite: ECANCELED & 500 error on bucket delete #12044

1 commit merged into from Feb 15, 2017

Conversation

Abhishekvrshny
Copy link

@cbodley
rgw: fix for bucket delete racing with mdlog sync
732405e 
Fixes: http://tracker.ceph.com/issues/17698

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit e9cf41f)
@Abhishekvrshny Abhishekvrshny changed the base branch from jewel to jewel-next November 17, 2016 14:39
@Abhishekvrshny Abhishekvrshny added this to the jewel milestone Nov 17, 2016
@Abhishekvrshny Abhishekvrshny self-assigned this Nov 17, 2016
ghost pushed a commit that referenced this pull request Nov 23, 2016
Merge pull request #12044: jewel: multisite: ECANCELED & 500 error on…
3060573 
… bucket delete

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Dec 5, 2016
Merge pull request #12044: jewel: multisite: ECANCELED & 500 error on…
f94a11c 
… bucket delete

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost changed the title jewel: multisite: ECANCELED & 500 error on bucket delete jewel: rgw: multisite: ECANCELED & 500 error on bucket delete Dec 5, 2016
@ghost ghost changed the base branch from jewel-next to jewel December 21, 2016 23:29
@smithfarm
Copy link
Contributor

Jenkins re-test this please

1 similar comment
@smithfarm
Copy link
Contributor

Jenkins re-test this please

@smithfarm
Copy link
Contributor

smithfarm commented Feb 1, 2017 •
edited

This passed rgw runs at http://tracker.ceph.com/issues/17851#note-50 and http://tracker.ceph.com/issues/17851#note-58 and is approved for merge after make check succeeds.

@smithfarm
Copy link
Contributor 

(11:46:45 AM) smithfarm: owasserm: thanks. For jewel integration rgw, then, what it comes down to is verifying that these 6 valgrind failures are all libtcmalloc-related: http://pulpito.front.sepia.ceph.com/smithfarm-2017-01-31_12:35:14-rgw-wip-jewel-backports-distro-basic-smithi/
(11:46:58 AM) smithfarm: owasserm: I will do that now
(11:47:05 AM) owasserm: smithfarm, thanks
(11:47:33 AM) smithfarm: owasserm: and assuming they are tcmalloc related, you said I can directly merge all the rgw PRs? Or do you want me to ask you for review in the PRs first?
(11:47:53 AM) owasserm: smithfarm, yes you can merge them
(11:48:19 AM) smithfarm: ok, will merge and do at least one or two more rgw runs before passing 10.2.6 to QE

@ghost ghost assigned yehudasa Feb 15, 2017
@ghost
Copy link

ghost commented Feb 15, 2017

@yehudasa does this backport look good to merge ? It passed the rgw suite at http://tracker.ceph.com/issues/17851#note-66.

@ghost ghost merged commit 9b4175e into ceph:jewel Feb 15, 2017
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@yehudasa yehudasa

@Abhishekvrshny Abhishekvrshny

Labels
bug-fix rgw
Projects
None yet
Milestone
jewel
4 participants
@Abhishekvrshny @smithfarm @yehudasa @cbodley