Skip to content

[8.19] [Observability] [Alerting] Fix alert tags when source tags is a string (#259729)#259766

Merged
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-259729
Mar 26, 2026
Merged

[8.19] [Observability] [Alerting] Fix alert tags when source tags is a string (#259729)#259766
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-259729

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

elastic#259729)

Resolves elastic#259504

When source documents have tags as a string instead of an array, the
spread syntax [...(additionalContext.tags ?? [])] iterates characters,
producing per-character tags on the alert (e.g. t, e, s, t, ...). This
PR normalizes tags to an array before merging with rule tags, fixing the
inventory threshold, metric threshold, and custom threshold executors.

(cherry picked from commit cd62e65)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Mar 26, 2026
@kibanamachine kibanamachine enabled auto-merge (squash) March 26, 2026 12:11
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-srcJs 3.6MB 3.6MB +127.0B

History

cc @benakansara

@kibanamachine kibanamachine merged commit 1eb6b52 into elastic:8.19 Mar 26, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Team:obs-ux-management

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants