Skip to content

Merge instance stats queries and use generic component

Ezekiel Kigbo requested to merge ek-merge-stats into master

What does this MR do?

This MR updates the generic instance stats query to use a shared graphl query for the:

  • Pipelines chart
  • Issues & Merge Requests

This builds on the work done in !45561 (merged).

Note: A follow up MR will migrate the 'Projects and groups' chart to use the same config style.

Related to #267538 (closed) Related to #268224 (closed)

Screenshots (strongly suggested)

Data available No data Single request failed Both requests failed
Screen_Shot_2020-10-28_at_7.53.56_pm Screen_Shot_2020-10-28_at_7.57.57_pm Screen_Shot_2020-10-28_at_7.52.24_pm Screen_Shot_2020-10-28_at_7.51.17_pm

Feature flag

Requires the instance_statistics feature flag

Testing

  1. Enable the feature flag
  2. Seed data FILTER=instance_statistics bundle exec rake db:seed_fu
  3. Sign in as an admin
  4. Navigate to /admin/instance_statistics

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 Ezekiel Kigbo

Merge request reports

Loading