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

secscan: split indexing of recent manifest into separate background operation (PROJQUAY-3501) #1281

Merged
merged 1 commit into from Apr 25, 2022

Conversation

kleesc
Copy link
Member

@kleesc kleesc commented Apr 22, 2022

Index recent manifests in a separate background process, allowing the
main process to correctly select random slabs from the entire table
set and marking them completed in the allocator (rbtree). This avoids
the worker having to start iterating from the beginning of the table
whenever it is restarted.

@kleesc kleesc requested review from syed and hdonnay April 22, 2022 20:38
@kleesc kleesc force-pushed the secscan-remove-batch-size-pagination branch from f7a1dd2 to 68acf9b Compare April 22, 2022 20:46
@kleesc kleesc requested a review from crozzy April 25, 2022 14:22
…peration (PROJQUAY-3501)

Index recent manifests in a separate background process, allowing the
main process to correctly select random slabs from the entire table
set and marking them completed in the allocator (rbtree). This avoids
the worker having to start iterating from the beginning of the table
whenever it is restarted.
@kleesc kleesc force-pushed the secscan-remove-batch-size-pagination branch from 68acf9b to 9474d8d Compare April 25, 2022 14:39
@kleesc kleesc merged commit ed77147 into quay:master Apr 25, 2022
@kleesc kleesc deleted the secscan-remove-batch-size-pagination branch April 25, 2022 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants