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: add support for the prefix parameter in account listing of Swift API #12258

Merged
merged 2 commits into from Feb 1, 2017
Merged

jewel: rgw: add support for the prefix parameter in account listing of Swift API #12258

merged 2 commits into from Feb 1, 2017

Conversation

rzarzynski
Copy link
Contributor

This is a backport of PR #12047.

Buckport ticket: http://tracker.ceph.com/issues/18098.

Both s3-tests and Tempest haven't found any regression here.

CC: @dachary, @yehudasa, @mattbenjamin.

@rzarzynski
rgw: optimize out ctor-copy in RGWListBuckets_ObjStore_SWIFT.
6ead4cc 
This commit has been backported to Jewel as a dependency for
0e0a14a.

Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit d68e625)
@rzarzynski
rgw: add support for the prefix parameter in account listing of Swift…
d9c1d86 
… API.

Fixes: http://tracker.ceph.com/issues/17931
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 0e0a14a)
@ghost ghost added this to the jewel milestone Dec 1, 2016
@ghost ghost self-assigned this Dec 1, 2016
ghost pushed a commit that referenced this pull request Dec 5, 2016
Merge pull request #12258: jewel: rgw: add support for the prefix par…
174835d 
…ameter in account listing of Swift API

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost changed the base branch from jewel-next to jewel December 21, 2016 23:27
@smithfarm smithfarm merged commit 056f8ac 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
feature rgw
Projects
None yet
Milestone
jewel
2 participants
@rzarzynski @smithfarm