Skip to content
Snippets Groups Projects

Add database sourced aggregated metric [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Merged Mikołaj Wawrzyniak requested to merge mwaw/add_database_sourced_aggregated_metric into master

What does this MR do?

This MR aims to wrap remaining pieces to complete implementation of &5227 (closed), in order to achieve this it introduces following changes:

  1. It adds first database sourced aggregated metric definition - this was done, to provide real use case for validation all changes in production env, as well as to discover some minor problems, with underlying implementation of aggregated metrics for database sourced metrics feature.
  2. It splits aggregated metrics definition for CE and EE, to avoid raising errors when metric in not present in EE

Relates to #300411 (closed) as it is impossible to verify if feature works correctly without any database sourced aggregate defined

Does this MR meet the acceptance criteria?

Conformity

Edited by Mikołaj Wawrzyniak

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
  • Mikołaj Wawrzyniak marked the checklist item Documentation (if required) as completed

    marked the checklist item Documentation (if required) as completed

  • Mikołaj Wawrzyniak changed the description

    changed the description

  • Mikołaj Wawrzyniak changed the description

    changed the description

  • added 1 commit

    • 846f2319 - Add database sourced aggregated metric

    Compare with previous version

  • Mikołaj Wawrzyniak changed the description

    changed the description

  • added 1 commit

    • cbae0e95 - Add database sourced aggregated metric

    Compare with previous version

  • Mikołaj Wawrzyniak changed the description

    changed the description

  • Mikołaj Wawrzyniak
  • added 1 commit

    • 04bc226a - Add database sourced aggregated metric

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading