Skip to content

[frontport] Wrap dashboard recording rule references in sum() for multi-validator compatibility (#5733)#5847

Open
ndr-ds wants to merge 1 commit intomainfrom
ndr-ds/_frontport_wrap_dashboard_recording_rule_references_in_sum_for_multi-validator_compatibility_5733_
Open

[frontport] Wrap dashboard recording rule references in sum() for multi-validator compatibility (#5733)#5847
ndr-ds wants to merge 1 commit intomainfrom
ndr-ds/_frontport_wrap_dashboard_recording_rule_references_in_sum_for_multi-validator_compatibility_5733_

Conversation

@ndr-ds
Copy link
Copy Markdown
Contributor

@ndr-ds ndr-ds commented Mar 27, 2026

Motivation

Recording rules produce single-valued results per instance, but when aggregating across multiple validators the raw recording rule name returns multiple series. Dashboard panels need to wrap them in sum().

Proposal

Wrap all recording rule references in dashboard panels with sum() for multi-validator compatibility.

Frontport of #5733.

Test Plan

CI

This was referenced Mar 27, 2026
Copy link
Copy Markdown
Contributor Author

ndr-ds commented Mar 27, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ndr-ds ndr-ds changed the base branch from ndr-ds/_frontport_optimize_grafana_dashboards_with_prometheus_recording_rules_5732_ to graphite-base/5847 March 27, 2026 16:48
@ndr-ds ndr-ds force-pushed the ndr-ds/_frontport_wrap_dashboard_recording_rule_references_in_sum_for_multi-validator_compatibility_5733_ branch from 26f5647 to 9b1dcf0 Compare March 27, 2026 16:48
@ndr-ds ndr-ds changed the base branch from graphite-base/5847 to ndr-ds/_frontport_optimize_grafana_dashboards_with_prometheus_recording_rules_5732_ March 27, 2026 18:22
@ndr-ds ndr-ds changed the base branch from ndr-ds/_frontport_optimize_grafana_dashboards_with_prometheus_recording_rules_5732_ to graphite-base/5847 March 27, 2026 22:41
@ndr-ds ndr-ds force-pushed the graphite-base/5847 branch from 8a3c50d to 1f09211 Compare March 27, 2026 22:41
@ndr-ds ndr-ds force-pushed the ndr-ds/_frontport_wrap_dashboard_recording_rule_references_in_sum_for_multi-validator_compatibility_5733_ branch from 34a1636 to 4985da4 Compare March 27, 2026 22:41
@ndr-ds ndr-ds changed the base branch from graphite-base/5847 to ndr-ds/_frontport_optimize_grafana_dashboards_with_prometheus_recording_rules_5732_ March 27, 2026 22:42
@ndr-ds ndr-ds changed the base branch from ndr-ds/_frontport_optimize_grafana_dashboards_with_prometheus_recording_rules_5732_ to graphite-base/5847 March 27, 2026 23:43
@ndr-ds ndr-ds force-pushed the graphite-base/5847 branch from 1f09211 to de03316 Compare March 27, 2026 23:43
@ndr-ds ndr-ds force-pushed the ndr-ds/_frontport_wrap_dashboard_recording_rule_references_in_sum_for_multi-validator_compatibility_5733_ branch from 4985da4 to ab940cb Compare March 27, 2026 23:43
@ndr-ds ndr-ds changed the base branch from graphite-base/5847 to ndr-ds/_frontport_optimize_grafana_dashboards_with_prometheus_recording_rules_5732_ March 27, 2026 23:43
@ndr-ds ndr-ds marked this pull request as ready for review March 28, 2026 00:35
@graphite-app graphite-app bot changed the base branch from ndr-ds/_frontport_optimize_grafana_dashboards_with_prometheus_recording_rules_5732_ to graphite-base/5847 March 29, 2026 00:25
@ndr-ds ndr-ds force-pushed the graphite-base/5847 branch from de03316 to 919fbaf Compare March 30, 2026 02:56
@ndr-ds ndr-ds force-pushed the ndr-ds/_frontport_wrap_dashboard_recording_rule_references_in_sum_for_multi-validator_compatibility_5733_ branch from ab940cb to b938894 Compare March 30, 2026 02:56
@graphite-app graphite-app bot changed the base branch from graphite-base/5847 to main March 30, 2026 02:56
@graphite-app
Copy link
Copy Markdown

graphite-app bot commented Mar 30, 2026

Merge activity

  • Mar 30, 2:56 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants