Skip to content

Metrics: Cleanup background workers & importers

Overview

Removal should take place in %16.0 or later.

Category:Metrics code is intertwined with the metrics GitLab reports on itself. Only Category:Metrics should be removed. Similarly, some items named like PrometheusAlert are also used for AlertManagement::Alert and should not be removed.

As part of Monitor: Metrics Deprecation and Removal (&10107 - closed), we want to delete the background processing tasks for the Metrics Dashboard pieces of Category:Metrics. This will include sidekiq workers & metrics importers.

Implementation plan

  1. Remove any migrations or triggers which start the import of system metrics or custom dashboards
  2. Remove classes corresponding to the import/background behavior
  3. Nullify relevant worker classes so they don't do any work
  4. Remove worker class files in next milestone

Relevant processes to follow

Relevant code

The list below is meant as an aid & may not be exhaustive.

NOTE: Some worker classes may be marked with Category:Metrics but are not connected to metrics dashboard and should not be removed. ex) GitlabPerformanceBarStatsWorker should not be removed.

Edited by Sarah Yasonik