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: Fix up to 1000 entries at a time in check_bad_index_multipart #16880

Merged
merged 1 commit into from Sep 7, 2017

Conversation

Projects
None yet
5 participants
@mattbenjamin
Contributor

mattbenjamin commented Aug 7, 2017

@mattbenjamin mattbenjamin added this to the jewel milestone Aug 7, 2017

@mattbenjamin mattbenjamin self-assigned this Aug 7, 2017

@mattbenjamin mattbenjamin requested a review from oritwas Aug 7, 2017

@mattbenjamin

This comment has been minimized.

Show comment
Hide comment
@mattbenjamin

mattbenjamin Aug 9, 2017

Contributor

@oritwas ping?

Contributor

mattbenjamin commented Aug 9, 2017

@oritwas ping?

@oritwas

oritwas approved these changes Aug 9, 2017

@vumrao

This comment has been minimized.

Show comment
Hide comment
@vumrao

vumrao Aug 9, 2017

Contributor

jewel backport tracker: http://tracker.ceph.com/issues/20902

Contributor

vumrao commented Aug 9, 2017

jewel backport tracker: http://tracker.ceph.com/issues/20902

rgw: Fix up to 1000 entries at a time in check_bad_index_multipart
Fixes: http://tracker.ceph.com/issues/20772

Signed-off-by: Orit Wasserman <owasserm@redhat.com>
(cherry picked from commit 25cef9a)
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
@mattbenjamin

This comment has been minimized.

Show comment
Hide comment
@mattbenjamin

mattbenjamin Aug 10, 2017

Contributor

@oritwas hi orit, I had a couple of howlers on here--could you rescan this? thx!

Contributor

mattbenjamin commented Aug 10, 2017

@oritwas hi orit, I had a couple of howlers on here--could you rescan this? thx!

@hansbogert

This comment has been minimized.

Show comment
Hide comment
@hansbogert

hansbogert Aug 21, 2017

Contributor

Will this be in the next minor release of Jewel?

Contributor

hansbogert commented Aug 21, 2017

Will this be in the next minor release of Jewel?

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 7, 2017

Contributor

@mattbenjamin Thanks for the backport! For the next time, we like it when there's a "Conflicts" section at the end explaining what the differences are between the backport and its counterpart in master.

(The "Conflicts" section is only needed for manual backports or ones that do not cherry-pick cleanly.)

Contributor

smithfarm commented Sep 7, 2017

@mattbenjamin Thanks for the backport! For the next time, we like it when there's a "Conflicts" section at the end explaining what the differences are between the backport and its counterpart in master.

(The "Conflicts" section is only needed for manual backports or ones that do not cherry-pick cleanly.)

@smithfarm smithfarm merged commit d5e7700 into ceph:jewel Sep 7, 2017

3 checks passed

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