Draft: Reduce urgency for caching bots access levels
What does this MR do and why?
- This MR sets a lower urgency for when a bot's
project_authorizationsneeds to be deleted. Since,DestroyUserWorkeralready has a lower urgency, this should be fine.
Context
- Project bots are created when a project access level token is created .
-
ResourceAccessTokens::InactiveTokensDeletionCronWorkercleans up these inactive project bots. To do this, it callsDestroyUserWorker. Both of these workers already have anurgency = low. -
DestroyUserWorker-> callsProjectRecalculatePerUserWorkerif the bot is aProjectMember, which finishes the job in time. -
DestroyUserWorker-> callsAuthorizedProjectsWorkerif the the bot is aGroupMember, so that the cache for all the projects in that namespace can be refreshed. This is where the jobs are consistently exceeding that target duration because often the number of records its deleting is > 10K. - Example correlation dashboard
MR acceptance checklist
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Edited by Hinam Mehra