Skip to content

ES query error should be classified as user error#226263

Merged
doakalexi merged 3 commits intoelastic:mainfrom
doakalexi:mark-timefield-error-as-user-error
Jul 14, 2025
Merged

ES query error should be classified as user error#226263
doakalexi merged 3 commits intoelastic:mainfrom
doakalexi:mark-timefield-error-as-user-error

Conversation

@doakalexi
Copy link
Copy Markdown
Contributor

@doakalexi doakalexi commented Jul 2, 2025

Fixes #221416

Summary

This PR updates the time field errors show up as user errors in the task metrics for ES query rules

Checklist

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

@doakalexi doakalexi changed the title Mark timefield error as user error ES query error should be classified as user error Jul 2, 2025
@doakalexi doakalexi added Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// backport:version Backport to applied version labels v9.2.0 release_note:skip Skip the PR/issue when compiling release notes labels Jul 2, 2025
@doakalexi doakalexi marked this pull request as ready for review July 2, 2025 21:27
@doakalexi doakalexi requested a review from a team as a code owner July 2, 2025 21:27
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

Copy link
Copy Markdown
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

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

LGTM

@doakalexi doakalexi merged commit 00cc4bd into elastic:main Jul 14, 2025
11 of 12 checks passed
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 16, 2025
@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226263 locally
cc: @doakalexi

1 similar comment
@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226263 locally
cc: @doakalexi

@doakalexi doakalexi removed the backport:version Backport to applied version labels label Jul 18, 2025
@kibanamachine kibanamachine added backport:skip This PR does not require backporting and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Jul 18, 2025
Bluefinger pushed a commit to Bluefinger/kibana that referenced this pull request Jul 22, 2025
Fixes elastic#221416

## Summary

This PR updates the time field errors show up as user errors in the task
metrics for ES query rules


### Checklist

- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
Fixes elastic#221416

## Summary

This PR updates the time field errors show up as user errors in the task
metrics for ES query rules


### Checklist

- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ES query error should be classified as user error

4 participants