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

Alerting: Fix queries and expressions in rule view details. #79497

Merged
merged 2 commits into from Dec 14, 2023

Conversation

soniaAguilarPeiron
Copy link
Member

What is this feature?

This PR fixes queries and expressions not being shown in alert view page.

The bug was introduced in this PR, where Stack component from @grafana/experimental in favor of @grafana/ui.

The recent update to the Stack component in @grafana/ui defaults to using nowrap. Consequently, some Stacks in our current view were automatically switched to nowrap, leading to the observed issue.

Special notes for your reviewer:

Before the change: (observe there is no D query)

before-queries-in-rule-view.mp4

After the change:

after-fix-queries-in-rule-view.mp4

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.

@soniaAguilarPeiron soniaAguilarPeiron added this to the 10.3.x milestone Dec 14, 2023
@soniaAguilarPeiron soniaAguilarPeiron requested a review from a team as a code owner December 14, 2023 09:14
@soniaAguilarPeiron soniaAguilarPeiron requested review from gillesdemey, VikaCep and konrad147 and removed request for a team December 14, 2023 09:14
@soniaAguilarPeiron soniaAguilarPeiron changed the title Alerting: fix queries and expressions in rule-view details. Alerting: Fix queries and expressions in rule-view details. Dec 14, 2023
@soniaAguilarPeiron soniaAguilarPeiron added the no-backport Skip backport of PR label Dec 14, 2023
@soniaAguilarPeiron soniaAguilarPeiron changed the title Alerting: Fix queries and expressions in rule-view details. Alerting: Fix queries and expressions in rule view details. Dec 14, 2023
@soniaAguilarPeiron soniaAguilarPeiron merged commit 6fffbe4 into main Dec 14, 2023
14 checks passed
@soniaAguilarPeiron soniaAguilarPeiron deleted the alerting/fix-queries-in-rule-view branch December 14, 2023 12:28
@summerwollin summerwollin modified the milestones: 10.3.x, 10.3.0 Jan 22, 2024
@konrad147 konrad147 added backport v10.2.x and removed no-backport Skip backport of PR labels Feb 16, 2024
grafana-delivery-bot bot pushed a commit that referenced this pull request Feb 16, 2024
* fix queries and expressions in rule-view details

* Add test

(cherry picked from commit 6fffbe4)
konrad147 pushed a commit that referenced this pull request Feb 16, 2024
…#82875)

Alerting: Fix queries and expressions in rule view details. (#79497)

* fix queries and expressions in rule-view details

* Add test

(cherry picked from commit 6fffbe4)

Co-authored-by: Sonia Aguilar <33540275+soniaAguilarPeiron@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants