Skip to content
Snippets Groups Projects

feat(sidekiq): move ProcessScanResultPolicyWorker to quarantine

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading