[Feature]: Add Trace Logs view to aggregate all span events by SoumyaRaikwar · Pull Request #3563 · jaegertracing/jaeger-ui

SoumyaRaikwar · 2026-03-05T21:06:35Z

Which problem is this PR solving?

Resolves [Feature]: View all logs nested under a trace or span #1377

Description of the changes

Added a new "Trace Logs" tab to the trace detail page that aggregates all log events from every span in a trace and displays them in a chronologically sorted table.
The table includes columns for Timestamp, Service , Span Name/Operation, Event/Log Name, Attributes, and Span ID (linked back to timeline view).
Supports both OTel terminology ("Trace Events", "Span Name", "Event") and legacy terminology ("Trace Logs", "Operation", "Log").
Shows an empty state message when the trace has no logs/events.
Added TraceLogs to ETraceViewType enum, tracking action, and menu entry in the view selector dropdown.

How was this change tested?

Added 9 new unit tests for the TraceLogsView component covering rendering, empty state, OTel/legacy terminology, attributes display, and span ID links.
Updated existing tests for AltViewOptions and TracePageHeader.track to cover the new view type.
All 2484 tests pass across 217 test suites.
All lint checks pass (prettier, tsc, eslint, license, copyright-year, tsx-naming).

Checklist

I have read https://github.com/jaegertracing/jaeger/blob/main/CONTRIBUTING_GUIDELINES.md
I have signed all commits
I have added unit tests for the new functionality
I have run lint and test steps successfully: make lint test

AI Usage in this PR (choose one)

See AI Usage Policy.

None: No AI tools were used in creating this PR
Light: AI provided minor assistance (formatting, simple suggestions)
Moderate: AI helped with code generation or debugging specific parts
Heavy: AI generated most or all of the code changes

Add a new 'Trace Logs' tab to the trace detail page that collects all log events from every span in a trace and displays them in a chronologically sorted table. The view includes columns for timestamp, service name, span name, event name, attributes, and a linked span ID. Supports both OTel and legacy terminology. Resolves jaegertracing#1377 Signed-off-by: SoumyaRaikwar <[email protected]>

Update AltViewOptions and TracePageHeader.track tests to cover the new TraceLogs view type menu item and tracking. Signed-off-by: SoumyaRaikwar <[email protected]>

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

yurishkuro · 2026-03-05T21:46:03Z

please add screenshots

SoumyaRaikwar · 2026-03-05T22:04:07Z

please add screenshots

@yurishkuro Here are the screenshots:
View selector dropdown with "Trace Logs" option:

Trace Logs view showing aggregated events from all spans, sorted chronologically:

yurishkuro · 2026-03-05T22:20:51Z

a few comments:

the layout seems to be very inefficient. The attributes column is arguably the most text-heavy but other columns consume too much real estate and should be less wide.
the naming is mixed up, Logs (OpenTracing), Attributes (OTEL). There is a config flag which tells the UI which nomenclature it should use (currently we support both old naming and OTEL terms)
the Attributes column should use the same component to display the tags as Span Detail panel, an expandable widget that renders one string in a collapsed form but expands into a table.
did you test in dark mode?

SoumyaRaikwar · 2026-03-05T22:45:57Z

a few comments:

the layout seems to be very inefficient. The attributes column is arguably the most text-heavy but other columns consume too much real estate and should be less wide.

the naming is mixed up, Logs (OpenTracing), Attributes (OTEL). There is a config flag which tells the UI which nomenclature it should use (currently we support both old naming and OTEL terms)

the Attributes column should use the same component to display the tags as Span Detail panel, an expandable widget that renders one string in a collapsed form but expands into a table.

did you test in dark mode?

@yurishkuro, thanks for catching dark mode i completely forgot that.

Column layout : Fixed column widths. Timestamp, Service Name, Operation, Log, and Span ID are narrow fixed-width columns. The Tags/Attributes column is flexible and takes the remaining space.
Naming consistency: Fixed. Now uses consistent terminology based on the useOtelTerms config flag:
- OpenTracing: "Trace Logs", "Operation", "Log", "Tags"
- OTel: "Trace Events", "Span Name", "Event", "Attributes"
Attributes column : Now uses the existing AccordionAttributes component from SpanDetail. Collapsed state shows a key=value summary, expanded state shows the full AttributesTable with copy buttons.
Dark mode : Tested and working. Screenshots below:

- Use AccordionAttributes expandable widget for attributes - Fix column widths for better layout efficiency - Consistent terminology: Tags/Attributes based on config flag - Verify dark mode compatibility Signed-off-by: SoumyaRaikwar <[email protected]>

Signed-off-by: Yuri Shkuro <[email protected]>

Copilot

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Signed-off-by: Soumya Raikwar <[email protected]>

…x.tsx Co-authored-by: Yuri Shkuro <[email protected]> Signed-off-by: Soumya Raikwar <[email protected]>

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Signed-off-by: Soumya Raikwar <[email protected]>

yurishkuro · 2026-03-06T03:55:17Z

please improve test coverage, see #3563 (comment)

Signed-off-by: SoumyaRaikwar <[email protected]>

SoumyaRaikwar · 2026-03-06T11:23:28Z

please improve test coverage, see #3563 (comment)

Done, improved in the latest commit.

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

yurishkuro · 2026-03-06T15:25:57Z

code coverage is still down

Signed-off-by: SoumyaRaikwar <[email protected]>

SoumyaRaikwar · 2026-03-06T20:00:48Z

code coverage is still down

sorry for it, i updated it.

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

SoumyaRaikwar added 2 commits March 6, 2026 02:31

test: update existing tests for Trace Logs view

3dd3830

Update AltViewOptions and TracePageHeader.track tests to cover the new TraceLogs view type menu item and tracking. Signed-off-by: SoumyaRaikwar <[email protected]>

Copilot AI review requested due to automatic review settings March 5, 2026 21:06

SoumyaRaikwar requested a review from a team as a code owner March 5, 2026 21:06

Copilot started reviewing on behalf of SoumyaRaikwar March 5, 2026 21:07 View session

Copilot AI reviewed Mar 5, 2026

View reviewed changes

github-actions Bot added the waiting-for-author PR is waiting for author to respond to maintainer's comments label Mar 5, 2026

github-actions Bot removed the waiting-for-author PR is waiting for author to respond to maintainer's comments label Mar 5, 2026

github-actions Bot added the waiting-for-author PR is waiting for author to respond to maintainer's comments label Mar 5, 2026

yurishkuro added the changelog:new-feature Change that should be called out as new feature in CHANGELOG label Mar 5, 2026

yurishkuro reviewed Mar 5, 2026

View reviewed changes

Comment thread packages/jaeger-ui/src/components/TracePage/TraceLogsView/index.tsx Outdated

Apply suggestion from @yurishkuro

f25ddd8

Signed-off-by: Yuri Shkuro <[email protected]>

Copilot AI review requested due to automatic review settings March 5, 2026 23:01

graphite-app Bot reviewed Mar 5, 2026

View reviewed changes

Comment thread packages/jaeger-ui/src/components/TracePage/TraceLogsView/index.tsx

github-actions Bot removed the waiting-for-author PR is waiting for author to respond to maintainer's comments label Mar 5, 2026

Copilot AI reviewed Mar 5, 2026

View reviewed changes

Comment thread packages/jaeger-ui/src/components/TracePage/TraceLogsView/index.test.js Outdated

Comment thread packages/jaeger-ui/src/components/TracePage/TraceLogsView/index.tsx

Update index.test.js

f3f36f2

Signed-off-by: Soumya Raikwar <[email protected]>

SoumyaRaikwar requested a review from yurishkuro March 5, 2026 23:35

yurishkuro reviewed Mar 6, 2026

View reviewed changes

Comment thread packages/jaeger-ui/src/components/TracePage/TraceLogsView/index.tsx Outdated

Update packages/jaeger-ui/src/components/TracePage/TraceLogsView/inde…

af3db90

…x.tsx Co-authored-by: Yuri Shkuro <[email protected]> Signed-off-by: Soumya Raikwar <[email protected]>

Copilot AI review requested due to automatic review settings March 6, 2026 00:49

Copilot started reviewing on behalf of SoumyaRaikwar March 6, 2026 00:49 View session

SoumyaRaikwar requested a review from yurishkuro March 6, 2026 00:49

Copilot AI reviewed Mar 6, 2026

View reviewed changes

Update index.css

212ab09

Signed-off-by: Soumya Raikwar <[email protected]>

github-actions Bot added the waiting-for-author PR is waiting for author to respond to maintainer's comments label Mar 6, 2026

test: improve code coverage for TraceLogsView

e9ae607

Signed-off-by: SoumyaRaikwar <[email protected]>

SoumyaRaikwar dismissed yurishkuro’s stale review via e9ae607 March 6, 2026 11:23

github-actions Bot removed the waiting-for-author PR is waiting for author to respond to maintainer's comments label Mar 6, 2026

Merge branch 'main' into feat/trace-logs-view

6cdd9a9

SoumyaRaikwar requested review from Copilot and yurishkuro March 6, 2026 11:23

Copilot started reviewing on behalf of SoumyaRaikwar March 6, 2026 11:24 View session

Copilot AI reviewed Mar 6, 2026

View reviewed changes

yurishkuro previously approved these changes Mar 6, 2026

View reviewed changes

yurishkuro enabled auto-merge March 6, 2026 14:44

github-actions Bot added the waiting-for-author PR is waiting for author to respond to maintainer's comments label Mar 6, 2026

test: fix toggle coverage for TraceLogsView

c8777f8

Signed-off-by: SoumyaRaikwar <[email protected]>

auto-merge was automatically disabled March 6, 2026 19:58
Head branch was pushed to by a user without write access

SoumyaRaikwar dismissed yurishkuro’s stale review via c8777f8 March 6, 2026 19:58

github-actions Bot removed the waiting-for-author PR is waiting for author to respond to maintainer's comments label Mar 6, 2026

SoumyaRaikwar requested a review from yurishkuro March 6, 2026 20:00

Merge branch 'main' into feat/trace-logs-view

bb786ff

Copilot AI review requested due to automatic review settings March 6, 2026 20:01

Copilot started reviewing on behalf of SoumyaRaikwar March 6, 2026 20:01 View session

Copilot AI reviewed Mar 6, 2026

View reviewed changes

yurishkuro approved these changes Mar 6, 2026

View reviewed changes

yurishkuro enabled auto-merge March 6, 2026 20:16

yurishkuro added this pull request to the merge queue Mar 6, 2026

Merged via the queue into jaegertracing:main with commit 2c1a2d1 Mar 6, 2026
18 of 19 checks passed

Conversation

SoumyaRaikwar commented Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Which problem is this PR solving?

Description of the changes

How was this change tested?

Checklist

AI Usage in this PR (choose one)

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

yurishkuro commented Mar 5, 2026

Uh oh!

SoumyaRaikwar commented Mar 5, 2026

Uh oh!

yurishkuro commented Mar 5, 2026

Uh oh!

SoumyaRaikwar commented Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

yurishkuro commented Mar 6, 2026

Uh oh!

SoumyaRaikwar commented Mar 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

yurishkuro commented Mar 6, 2026

Uh oh!

SoumyaRaikwar commented Mar 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

SoumyaRaikwar commented Mar 5, 2026 •

edited

Loading

SoumyaRaikwar commented Mar 5, 2026 •

edited

Loading