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: bucket check remove _multipart_ prefix #11470

Merged
merged 1 commit into from Feb 1, 2017
Merged

jewel: rgw: bucket check remove _multipart_ prefix #11470

merged 1 commit into from Feb 1, 2017

Conversation

ghost
Copy link

@ghost ghost commented Oct 13, 2016

@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
… prefix

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

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

ghost commented Oct 20, 2016

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

@ghost ghost assigned yehudasa Oct 20, 2016
@ghost ghost changed the base branch from jewel to jewel-next November 9, 2016 09:58
Fixes: 13724

Signed-off-by: Weijun Duan <duanweijun@h3c.com>
(cherry picked from commit e55c9c8)
@ghost
Copy link
Author

ghost commented Nov 9, 2016

jenkins test this please (bluefs core)

@ghost
Copy link
Author

ghost commented Nov 9, 2016

jenkins test this please (general jenkins failure)

ghost pushed a commit that referenced this pull request Nov 9, 2016
… prefix

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

ghost commented Nov 11, 2016

@yehudasa ping

ghost pushed a commit that referenced this pull request Nov 23, 2016
… prefix

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

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

ghost commented Dec 5, 2016

@yehudasa ping

@ghost ghost changed the base branch from jewel-next to jewel December 21, 2016 23:32
@ghost
Copy link
Author

ghost commented Dec 21, 2016

check this please

@smithfarm
Copy link
Contributor

@smithfarm smithfarm merged commit a61be36 into ceph:jewel Feb 1, 2017
@smithfarm smithfarm changed the title jewel: rgw:bucket check remove _multipart_ prefix jewel: rgw: bucket check remove _multipart_ prefix Feb 1, 2017
@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants