Skip to content
Snippets Groups Projects

Add metrics for paused indexing Sidekiq queue size

All threads resolved!

What does this MR do?

Issue #217918 (closed)

  • Add a new Prometheus metric for the index paused Sidekiq queue size. Following the example set by the existing metrics.
  • Updated documentation
  • Updated unit test

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Terri Chu

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
  • Micaël Bergeron approved this merge request

    approved this merge request

  • @terrichu I'm glad you decided to do this, as I remember you talking about that just yesterday, so props to Results here.

    I only have a single comment on the naming, feel free to apply it or not.

    I approve this change.

  • Achilleas Pipinellis approved this merge request

    approved this merge request

  • Terri Chu resolved all threads

    resolved all threads

  • Terri Chu added 1 commit

    added 1 commit

    • 2ca03b10 - fixup! Add Prometheus Metrics for Index Pause Sidekiq Queue

    Compare with previous version

  • Terri Chu marked as a Work In Progress from 2ca03b10

    marked as a Work In Progress from 2ca03b10

  • Author Maintainer

    @mayra-cabrera are you available for a back end maintainer review?

  • Terri Chu unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Mayra Cabrera approved this merge request

    approved this merge request

  • Mayra Cabrera enabled an automatic merge when the pipeline for f9236a3e succeeds

    enabled an automatic merge when the pipeline for f9236a3e succeeds

  • Thanks @terrichu! This LGTM, MWPS set :rocket:

  • merged

  • Mayra Cabrera mentioned in commit fd57cd63

    mentioned in commit fd57cd63

  • added workflowstaging label and removed workflowin review label

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Dylan Griffith mentioned in commit 9ab08d67

    mentioned in commit 9ab08d67

  • Dylan Griffith mentioned in merge request !36395 (merged)

    mentioned in merge request !36395 (merged)

  • Terri Chu mentioned in merge request !36473 (merged)

    mentioned in merge request !36473 (merged)

  • added devopsdata stores label and removed devopssystems label

  • Please register or sign in to reply
    Loading