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: cls/rgw: don't assert in decode_list_index_key() #24391

Merged
merged 1 commit into from Oct 4, 2018

Conversation

Projects
None yet
4 participants
@smithfarm
Contributor

smithfarm commented Oct 3, 2018

cls/rgw: don't assert in decode_list_index_key()
Fixes: http://tracker.ceph.com/issues/24117

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit e71fba0)

Conflicts:
	src/cls/rgw/cls_rgw.cc
- use original non-C++17 version of escape_str() function in luminous

@smithfarm smithfarm added the rgw label Oct 3, 2018

@smithfarm smithfarm added this to the luminous milestone Oct 3, 2018

@smithfarm smithfarm requested a review from cbodley Oct 3, 2018

@smithfarm smithfarm changed the title from cls/rgw: don't assert in decode_list_index_key() to luminous: cls/rgw: don't assert in decode_list_index_key() Oct 3, 2018

@yuriw

This comment has been minimized.

Contributor

yuriw commented Oct 3, 2018

@cbodley

cbodley approved these changes Oct 4, 2018

@yuriw yuriw merged commit 102ef83 into ceph:luminous Oct 4, 2018

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

@smithfarm smithfarm deleted the smithfarm:wip-24630-luminous branch Oct 5, 2018

@smithfarm smithfarm changed the title from luminous: cls/rgw: don't assert in decode_list_index_key() to luminous: rgw: cls/rgw: don't assert in decode_list_index_key() Oct 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment