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

Jump to bottom

jewel: rgw: multisite upgrade from hammer -> jewel ignores rgw_region_root_pool #12156

Merged
merged 2 commits into from Feb 1, 2017
Merged

jewel: rgw: multisite upgrade from hammer -> jewel ignores rgw_region_root_pool #12156

merged 2 commits into from Feb 1, 2017

Conversation

ghost
Copy link

@ghost ghost commented Nov 23, 2016

@cbodley @ldachary
rgw: region conversion respects pre-existing rgw_region_root_pool
af10e57 
Fixes: http://tracker.ceph.com/issues/17963

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 8b70dc6)
@cbodley @ldachary
rgw: look for region_map in rgw_region_root_pool
a0b4e60 
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit b81d880)
@ghost ghost self-assigned this Nov 23, 2016
@ghost ghost added this to the jewel milestone Nov 23, 2016
@ghost ghost added bug-fix rgw labels Nov 23, 2016
@ghost ghost changed the base branch from jewel to jewel-next November 23, 2016 08:16
ghost pushed a commit that referenced this pull request Nov 23, 2016
Merge pull request #12156: jewel: multisite upgrade from hammer -> je…
1b5874f 
…wel ignores rgw_region_root_pool

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Dec 5, 2016
Merge pull request #12156: jewel: multisite upgrade from hammer -> je…
5df3f4f 
…wel ignores rgw_region_root_pool

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost changed the base branch from jewel-next to jewel December 21, 2016 23:27
@ghost
Copy link
Author

ghost commented Dec 21, 2016

check this please

@smithfarm smithfarm changed the title jewel: multisite upgrade from hammer -> jewel ignores rgw_region_root_pool jewel: rgw: multisite upgrade from hammer -> jewel ignores rgw_region_root_pool Feb 1, 2017
@smithfarm smithfarm merged commit 41fcf74 into ceph:jewel Feb 1, 2017
@smithfarm
Copy link
Contributor

This passed rgw runs at http://tracker.ceph.com/issues/17851#note-50 and http://tracker.ceph.com/issues/17851#note-58

@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
Reviewers
No reviews
Assignees
No one assigned
Labels
bug-fix rgw
Projects
None yet
Milestone
jewel
2 participants
@smithfarm @cbodley