Skip to content

WIP: Setup multiple queries table & interface - Part 1

Sarah Yasonik requested to merge setup-multiple-queries into master

What does this MR do?

Part 1 of supporting multiple queries per chart. Part 1 blocks Part 2, which blocks Parts 3A & 3B.

Part 1: setup-multiple-queries

  • add new prometheus_queries table
  • add new class, factory, specs

Part 2: import-multiple-queries

  • migrate existing query data from metrics table to queries table
  • update PrometheusMetric#queries to return contents of queries table
  • update importer to expect multiple queries

Part 3A: display-multiple-queries

  • update common_metrics.yml to merge target charts & call updated importer (TBD)
  • any FE work (TBD)

Part 3B: cleanup-multiple-queries

  • "ignore" query columns in metrics table
  • delete query columns in post-deployment migration

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/54877

Does this MR meet the acceptance criteria?

Merge request reports