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

rgw: auto-clean reshard queue entries for non-existent buckets #31323

Merged
merged 1 commit into from Nov 10, 2019

Conversation

ivancich
Copy link
Member

@ivancich ivancich commented Nov 1, 2019

It is possible for a bucket to be added to the reshard queue and then to be removed before its entry in the reshard queue is processed. Prior to this pr, the entry would remain on the reshard queue and stop the queue from being processed further. With this pr, the entry will be automatically removed with appropriate log entries and processing of the queue can continue.

Tracker: https://tracker.ceph.com/issues/42595

It is possible for a bucket to be added to the reshard queue and then
to be removed before its entry in the reshard queue is processed. When
this is now encountered, processing of the reshard queue errors out.

This fix recognizes when the reshard queue entry refers to a
non-existent bucket and remove the entry from the reshard queue,
allowing processing of the queue to continue.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>
Copy link
Contributor

@mattbenjamin mattbenjamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ivancich ivancich force-pushed the wip-autoclean-reshard-long-entries branch from 8c156d7 to 304ad88 Compare November 1, 2019 19:01
@ivancich ivancich added the wip-eric-testing-1 for ivancich testing label Nov 9, 2019
@ivancich
Copy link
Member Author

s3-tests shows no regressions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants