Skip to content

[9.1] [Discover][APM] Fix summary column badge field filtering for ES|QL (#226598)#227038

Merged
kibanamachine merged 1 commit intoelastic:9.1from
kibanamachine:backport/9.1/pr-226598
Jul 8, 2025
Merged

[9.1] [Discover][APM] Fix summary column badge field filtering for ES|QL (#226598)#227038
kibanamachine merged 1 commit intoelastic:9.1from
kibanamachine:backport/9.1/pr-226598

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

…lastic#226598)

## Summary

Fixes the filter buttons for the summary badge icons on the traces
document source profile, so that in ES|QL mode, the filters correctly
add the field query without defaulting to casting the field value to a
string.

Closes elastic#226414

![image](https://github.com/user-attachments/assets/c306d74f-84c1-4afb-aeed-a29509d389ec)

## How to test

- Enable the Observability mode on the space you are in, navigate to
Discover and enable the ES|QL mode
- Query for a `traces-*` index, `remote_cluster:traces-*` if using
edge-oblt-ccs.
- On the summary column, click on a badge icon and add a filter to the
query.
- The field value should be in its assumed format and not being casted,
such as a `span.duration.us` should be as a `number`, `service.name` as
a `string`.

(cherry picked from commit 53219ed)
@kibanamachine kibanamachine merged commit 8d4e9dd into elastic:9.1 Jul 8, 2025
16 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #63 / Journey[infra_hosts_view] "before all" hook for "Navigate to Hosts view and load 500 hosts"

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/discover-contextual-components 36 37 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
discover 1.1MB 1.1MB +71.0B
Unknown metric groups

API count

id before after diff
@kbn/discover-contextual-components 38 39 +1

cc @Bluefinger

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants