Skip to content

Add report caching for license rules sync

Igor Frenkel requested to merge 398679-add-report-caching into master

What does this MR do and why?

Security::SyncLicenseScanningRulesService is invoked by Ci::SyncReportsToReportApprovalRulesWorker which is a frequently executing worker. The service makes an expensive call to the database each time it is executed causing high utilization. See checkup for more info.

Caching this report would be very useful. Based on log data (internal link) this is a very frequently called worker.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #398679 (closed)

Merge request reports