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

luminous: rgw: list_objects() honors end_marker regardless of namespace. #17832

Merged
merged 1 commit into from Oct 14, 2017

Conversation

Projects
None yet
4 participants
@rzarzynski
Contributor

rzarzynski commented Sep 20, 2017

http://tracker.ceph.com/issues/21453

This patch fixes a regression related to handling of the end_marker parameter during Swift's container listing operation. It has been introduced in a5d1fa0 and causes Tempest's test_list_container_contents_with_end_marker to fail.

Fixes: http://tracker.ceph.com/issues/18977
Signed-off-by: Radoslaw Zarzynski rzarzynski@mirantis.com
(cherry picked from commit 09db1e4)


This is the luminous backport of PR #15273.

rgw: list_objects() honors end_marker regardless of namespace.
This patch fixes a regression related to handling of the end_marker
parameter during Swift's container listing operation. It has been
introduced in a5d1fa0 and causes
Tempest's test_list_container_contents_with_end_marker to fail.

Fixes: http://tracker.ceph.com/issues/18977
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 09db1e4)
@mattbenjamin

This comment has been minimized.

Show comment
Hide comment
@mattbenjamin

mattbenjamin Sep 20, 2017

Contributor

@rzarzynski iirc this looks ok, need a suite run :)

Contributor

mattbenjamin commented Sep 20, 2017

@rzarzynski iirc this looks ok, need a suite run :)

@rzarzynski

This comment has been minimized.

Show comment
Hide comment
@rzarzynski

rzarzynski Sep 20, 2017

Contributor

jenkins retest this please (run-cli-tests)

Contributor

rzarzynski commented Sep 20, 2017

jenkins retest this please (run-cli-tests)

@smithfarm smithfarm added this to the luminous milestone Sep 20, 2017

@rzarzynski

This comment has been minimized.

Show comment
Hide comment
@mdw-at-linuxbox

Looks good to me. The errors I see in jenkins I think are unrelated weirdness, nothing to do with this change.

@mdw-at-linuxbox mdw-at-linuxbox self-assigned this Oct 14, 2017

@mdw-at-linuxbox mdw-at-linuxbox merged commit c4ac0d6 into ceph:luminous Oct 14, 2017

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment