Skip to content

[Alerting][Custom Threshold Rule] Update custom_threshold_rule schema and test snapshot#249998

Merged
miguelmartin-elastic merged 1 commit intoelastic:mainfrom
miguelmartin-elastic:231158-Intermediate-Add-kql-filtering-custom-rule
Jan 23, 2026
Merged

[Alerting][Custom Threshold Rule] Update custom_threshold_rule schema and test snapshot#249998
miguelmartin-elastic merged 1 commit intoelastic:mainfrom
miguelmartin-elastic:231158-Intermediate-Add-kql-filtering-custom-rule

Conversation

@miguelmartin-elastic
Copy link
Copy Markdown
Contributor

@miguelmartin-elastic miguelmartin-elastic commented Jan 22, 2026

Summary

This PR is for an intermediate release for #248845 .

  • Update the Custom Threshold Rule schema to accept an optional KQL string filter as part of the metrics items : schema changed from never to optional string

Test

I tested it manually and verified that Custom Threshold Rules with the new schema in place are executed without issues in current Kibana version in main

@miguelmartin-elastic
Copy link
Copy Markdown
Contributor Author

/ci

@github-actions github-actions Bot added the author:actionable-obs PRs authored by the actionable obs team label Jan 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

@miguelmartin-elastic, it looks like you're updating the parameters for a rule type!

Please review the guidelines for making additive changes to rule type parameters and determine if your changes require an intermediate release.

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

@miguelmartin-elastic miguelmartin-elastic added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Jan 23, 2026
@miguelmartin-elastic miguelmartin-elastic marked this pull request as ready for review January 23, 2026 09:55
@miguelmartin-elastic miguelmartin-elastic requested a review from a team as a code owner January 23, 2026 09:55
Copy link
Copy Markdown
Contributor

@pmuellr pmuellr left a comment

Choose a reason for hiding this comment

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

LGTM. Hopefully this will be applied to serverless on Monday / Tuesday, so you can then commit changes which set the new field. In any case, this PR's code needs to be running in Serverless the release BEFORE the one that sets the new field.

@miguelmartin-elastic miguelmartin-elastic merged commit 6f03fc5 into elastic:main Jan 23, 2026
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author:actionable-obs PRs authored by the actionable obs team backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants