Skip to content

Replace find with find_by_id in SyncScanPoliciesWorker

Sashi Kumar Kumaresan requested to merge sk/fix-retries into master

What does this MR do and why?

This MR fixes a bug in Security::SyncScanPoliciesWorker where ActiveRecord::RecordNotFound exception was raised when configuration_id was deleted. This results in unwanted retry by sidekiq and taking a toll on our error budget.

Logs

This MR uses find_by_id as suggested in docs.

MR acceptance checklist

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

Edited by Sashi Kumar Kumaresan

Merge request reports