Skip to content

Remove `StoreSecurityReportsWorker` class

  • Please check this box if this contribution uses AI-generated content (including content generated by GitLab Duo features) as outlined in the GitLab DCO & CLA

What does this MR do and why?

This merge request addresses the removal of the deprecated StoreSecurityReportsWorker class and updates the associated Sidekiq queue configuration files and Migration has been added to remove any scheduled instances. Specifically, it includes the following changes:

  • Removal of StoreSecurityReportsWorker class:

    • The StoreSecurityReportsWorker class file has been removed as it is no longer needed.
  • Regeneration of Sidekiq queue configuration:

    • The Sidekiq queue configuration files have been regenerated to ensure that the removal of the StoreSecurityReportsWorker is properly reflected in the system.
  • Database migration:

    • A new database migration has been added to remove any scheduled instances of the deprecated StoreSecurityReportsWorker from Sidekiq. This ensures that there are no orphaned jobs left in the queue.

Issue: #467944 (closed)

Screenshots or screen recordings

Before After

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

  1. Run rails db:migrate to apply the migration.
  2. Verify that the StoreSecurityReportsWorker class file has been removed.
  3. Check the Sidekiq queue configuration to confirm changes.

Milestone

This MR is targeted for milestone %17.3.

Edited by Sahil Sarawagi

Merge request reports

Loading