Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v10.0.x] Alerting: Exclude expression refIDs from NoData state #72394

Merged
merged 2 commits into from
Jul 26, 2023

Conversation

grafana-delivery-bot[bot]
Copy link
Contributor

Backport 5ba164d from #72219


What is this feature?
Fixes alerting evaluator to not include expression RefID to NoData result

Why do we need this feature?
This removes an extra state that mentions refID of expressions when no data result is executed as NoData state.

Example

firefox_kUGehpZzsH

Who is this feature for?
Those who configure execution of NoData as NoData.

Which issue(s) does this PR fix?:

Fixes #71993

Special notes for your reviewer:

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@grafana-delivery-bot grafana-delivery-bot bot added this to the 10.0.x milestone Jul 26, 2023
@grafana-delivery-bot grafana-delivery-bot bot added add to changelog area/alerting Grafana Alerting area/backend backport A backport PR pr/external This PR is from external contributor type/bug labels Jul 26, 2023
@yuri-tceretian yuri-tceretian merged commit f848c9d into v10.0.x Jul 26, 2023
7 checks passed
@yuri-tceretian yuri-tceretian deleted the backport-72219-to-v10.0.x branch July 26, 2023 16:45
@zerok zerok modified the milestones: 10.0.x, 10.0.4 Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/alerting Grafana Alerting area/backend backport A backport PR pr/external This PR is from external contributor type/bug
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants