Skip to content

Add reason to query stats. Add query too expensive to qfe reasons#7479

Open
eeldaly wants to merge 3 commits intocortexproject:masterfrom
eeldaly:qfe-logs
Open

Add reason to query stats. Add query too expensive to qfe reasons#7479
eeldaly wants to merge 3 commits intocortexproject:masterfrom
eeldaly:qfe-logs

Conversation

@eeldaly
Copy link
Copy Markdown
Contributor

@eeldaly eeldaly commented May 5, 2026

What this PR does:

  • Adds reason field to query stats
  • Adds query_too_expensive to QFE reasons and cortex_rejected_queries_total metric

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]
  • docs/configuration/v1-guarantees.md updated if this PR introduces experimental flags

Signed-off-by: Essam Eldaly <eeldaly@amazon.com>
eeldaly added 2 commits May 5, 2026 12:17
Signed-off-by: Essam Eldaly <eeldaly@amazon.com>
Signed-off-by: Essam Eldaly <eeldaly@amazon.com>
@pull-request-size pull-request-size Bot added size/M and removed size/S labels May 5, 2026
Copy link
Copy Markdown
Member

@friedrichg friedrichg left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 5, 2026
Copy link
Copy Markdown
Contributor

@justinjung04 justinjung04 left a comment

Choose a reason for hiding this comment

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

nice, thank you!

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

Labels

component/query-frontend lgtm This PR has been approved by a maintainer size/M type/feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants