Skip to content

feat(sidekiq): move ProcessScanResultPolicyWorker to quarantine

Steve Xuereb requested to merge feat/move-proccess-scan-result into master

What does this MR do?

What

Move the worker Security::ProcessScanResultPolicyWorker from database_throttled to quarantine

Why

Security::ProcessScanResultPolicyWorker was moved to database_throtted for performance issues in !3242 (merged), this was fixed in gitlab-org/gitlab!137707 (merged).

Moving this to quarantine will allow us to run more jobs and validate that we are not hammering the database.

Reference: gitlab-com/gl-infra/production#17168 (closed)

Author Check-list

Please read the Contributing document and once you do, complete the following:

  • Check if all of the following apply:
    • Assign to the correct reviewer per the contributing document
    • Apply the correct metadata per the contributing document
    • Link to related MRs for applying the changes on other environments
    • Link to related Chef changes
    • If necessary link to a Criticality 4 Change Request issue

Reviewer Check-list

  • Check if all of the following apply:
    • Reviewed the diff jobs to confirm changes are as expected
    • No changes shown in the diffs not associated with this MR - This may require a rebase or further investigation

Applier Check-list

  • Make sure there is no ongoing deployment for the affected envs before merging (see #announcements slack channel)

Merge request reports