Skip to content

chore(alerting): Gate alertingQueryAndExpressionsStepMode FT on Grafana version#2758

Closed
konstantinmv wants to merge 4 commits into
mainfrom
alerting/remove-step-mode-ft
Closed

chore(alerting): Gate alertingQueryAndExpressionsStepMode FT on Grafana version#2758
konstantinmv wants to merge 4 commits into
mainfrom
alerting/remove-step-mode-ft

Conversation

@konstantinmv

Copy link
Copy Markdown

What

Update AlertRuleEditPage.isAdvancedModeSupported() to account for the removal of the alertingQueryAndExpressionsStepMode feature toggle in Grafana 13.2.0.

Why

In Grafana 13.2.0 the toggle was removed and the advanced query/expression mode switch became always available (GA).

##How

Advanced mode is now considered supported when either:

  • Grafana >= 13.2.0 (toggle gone, feature always on), or
  • the alertingQueryAndExpressionsStepMode toggle is enabled (Grafana < 13.2.0)

Follow-up to grafana/grafana#125086.

@grafana-catalog-project-bot grafana-catalog-project-bot Bot moved this from 📬 Triage to 🧑‍💻 In development in Grafana Catalog Team Jul 8, 2026
@konstantinmv konstantinmv marked this pull request as ready for review July 8, 2026 09:05
@konstantinmv konstantinmv requested a review from a team as a code owner July 8, 2026 09:05
@konstantinmv konstantinmv requested review from Ukochka, academo and xnyo July 8, 2026 09:05

@konrad147 konrad147 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@leventebalogh leventebalogh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@tolzhabayev

Copy link
Copy Markdown
Collaborator

Slightly alternative / more comprehensive fix #2762

@konstantinmv

Copy link
Copy Markdown
Author

Slightly alternative / more comprehensive fix #2762

Yes, that's much better, thank you for taking the time to open it! I spent some time yesterday looking into the nighlty failure but couldn't figure it out that quick, thanks again for this.
Closing this PR in favour of the one you've opened.

@github-project-automation github-project-automation Bot moved this from 🔬 In review to 🚀 Shipped in Grafana Catalog Team Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🚀 Shipped

Development

Successfully merging this pull request may close these issues.

4 participants