Skip to content

[8.19] [Discover][Traces] Fix processed OTel waterfall flyout (#224548)#224574

Merged
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-224548
Jun 20, 2025
Merged

[8.19] [Discover][Traces] Fix processed OTel waterfall flyout (#224548)#224574
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-224548

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

## Summary

Relates to elastic#208707

While reviewing the Traces in Discover feature, we noticed a couple of
issues affecting processed OTel data in the waterfall flyout:

- The data wasn’t properly formatted because we were pulling it directly
from `_source`, which meant the UI wasn’t receiving the processed
version. Now we're using `fields` the same way it's used in Discover.
- The logic used to set the flyout title (to determine whether the
document was a span or a transaction) relied entirely on the presence of
`parent.id`. However, since transactions can also be children of other
transactions, we’ve added an extra check using `transaction.name` (spans
reference the parent transaction via `transaction.id`) to ensure we're
correctly identifying spans.

||Discover|Transaction|Span|
|-|-|-|-|
|Before|![Screenshot 2025-06-19 at 12 48
32](https://github.com/user-attachments/assets/7a2f7db9-c348-40b7-a974-c57436db70f1)|![Screenshot
2025-06-19 at 12 50
02](https://github.com/user-attachments/assets/18098108-12e9-4ed0-8fae-f9653729272b)|![Screenshot
2025-06-19 at 12 49
47](https://github.com/user-attachments/assets/6aa120b8-ba0f-48e7-b775-ab8101849635)|
|After|Same than before.|![Screenshot 2025-06-19 at 12 51
11](https://github.com/user-attachments/assets/3e29bd8a-d803-4efd-8bde-7baa63f25067)|![Screenshot
2025-06-19 at 12 47
52](https://github.com/user-attachments/assets/f82bc2a9-02a8-4a98-94ba-1449e8588d78)|

(cherry picked from commit e0a4b63)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 19, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) June 19, 2025 13:10
@kibanamachine kibanamachine merged commit f5e0266 into elastic:8.19 Jun 20, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
unifiedDocViewer 348 349 +1

Async chunks

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

id before after diff
unifiedDocViewer 239.2KB 239.3KB +84.0B

History

cc @iblancof

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