Skip to content

[8.x] [Discover] Exclude Elasticsearch metadata fields from Display in Content Column (#213255)#215132

Merged
rStelmach merged 5 commits intoelastic:8.xfrom
rStelmach:backport/8.x/pr-213255
Mar 24, 2025
Merged

[8.x] [Discover] Exclude Elasticsearch metadata fields from Display in Content Column (#213255)#215132
rStelmach merged 5 commits intoelastic:8.xfrom
rStelmach:backport/8.x/pr-213255

Conversation

@rStelmach
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…ent Column (elastic#213255)

Removed "_" from fields to exclude in the Display and replaced it with
Elasticsearch's metadata fields.

(cherry picked from commit 49ebf9e)

# Conflicts:
#	src/platform/packages/shared/kbn-discover-contextual-components/src/data_types/logs/components/summary_column/utils.tsx
@rStelmach rStelmach added the backport This PR is a backport of another PR label Mar 19, 2025
@rStelmach rStelmach enabled auto-merge (squash) March 19, 2025 10:53
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

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 38 +2
@kbn/discover-utils 238 239 +1
total +3

Async chunks

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

id before after diff
discover 1003.5KB 1003.8KB +257.0B
Unknown metric groups

API count

id before after diff
@kbn/discover-contextual-components 39 41 +2
@kbn/discover-utils 288 289 +1
total +3

History

Copy link
Contributor

@yngrdyn yngrdyn left a comment

Choose a reason for hiding this comment

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

Didn't review the changes, just approving manual backport

@rStelmach rStelmach merged commit 0224580 into elastic:8.x Mar 24, 2025
7 of 8 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants