Skip to content

🌊 Streams: Adjust default mappings for wired streams#228618

Merged
flash1293 merged 4 commits intoelastic:mainfrom
flash1293:flash1293/adjust-wired-streams-mapping
Jul 23, 2025
Merged

🌊 Streams: Adjust default mappings for wired streams#228618
flash1293 merged 4 commits intoelastic:mainfrom
flash1293:flash1293/adjust-wired-streams-mapping

Conversation

@flash1293
Copy link
Contributor

This fine tunes the mappings for wired streams:

  • No flattened field type for body.structured - instead it behaves like *.attributes
  • observed_timestamp isn't mapped by default (can be mapped if the user wishes to)
  • resource.attributes.service.name is mapped by default

@flash1293 flash1293 added the release_note:skip Skip the PR/issue when compiling release notes label Jul 18, 2025
@flash1293 flash1293 requested a review from a team as a code owner July 18, 2025 14:08
@flash1293 flash1293 added backport:skip This PR does not require backporting Team:obs-onboarding Observability Onboarding Team Feature:Streams This is the label for the Streams Project v9.2.0 labels Jul 18, 2025
@elasticmachine
Copy link
Contributor

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

Comment on lines -94 to -117
text: {
type: 'match_only_text',
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't see the deletion of body.text in the summary, double checking this is expected ?

@flash1293
Copy link
Contributor Author

flash1293 commented Jul 22, 2025

@klacabane A sorry, forgot about this - it was duplicated since it's spread in via

anyway, since body.text is part of the baseFields:

@flash1293 flash1293 requested a review from a team as a code owner July 23, 2025 07:55
Copy link
Contributor

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

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

LGTM, I will fix codeowners in the follow-up so we don't block you next time

@flash1293 flash1293 enabled auto-merge (squash) July 23, 2025 07:59
@flash1293 flash1293 merged commit 1fa6f03 into elastic:main Jul 23, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
This fine tunes the mappings for wired streams:
* No flattened field type for `body.structured` - instead it behaves
like `*.attributes`
* `observed_timestamp` isn't mapped by default (can be mapped if the
user wishes to)
* `resource.attributes.service.name` is mapped by default
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Jul 31, 2025
This fine tunes the mappings for wired streams:
* No flattened field type for `body.structured` - instead it behaves
like `*.attributes`
* `observed_timestamp` isn't mapped by default (can be mapped if the
user wishes to)
* `resource.attributes.service.name` is mapped by default
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 Feature:Streams This is the label for the Streams Project release_note:skip Skip the PR/issue when compiling release notes Team:obs-onboarding Observability Onboarding Team v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants