Skip to content

Correctly display events and links for table view queries - #1345

Merged
adamyeats merged 3 commits into
grafana:mainfrom
mcorbin:fix-table-events-link
Sep 4, 2025
Merged

adamyeats merged 3 commits into
grafana:mainfrom
mcorbin:fix-table-events-link

Conversation

@mcorbin

@mcorbin mcorbin commented Aug 19, 2025

Copy link
Copy Markdown
Contributor

Fix #1262

Pass the events and links configuration to the query and use otel columns if opentelemetry is enabled.

This allows displaying events in the trace panel.

I tested it locally: the events are correctly displayed when coming from a table trace ID.

@CLAassistant

CLAassistant commented Aug 19, 2025 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@mcorbin
mcorbin force-pushed the fix-table-events-link branch from 32dcac7 to 0004bd6 Compare August 19, 2025 12:56
@bossinc bossinc moved this from Incoming to Needs Review in Partner Datasources Aug 22, 2025
Fix grafana#1262

Pass the events and links configuration to the query and use otel
columns if opentelemetry is enabled.

This allows displaying events to the trace panel.
@adamyeats
adamyeats force-pushed the fix-table-events-link branch from 60a9e19 to 52f9e87 Compare August 29, 2025 17:25
adamyeats
adamyeats previously approved these changes Sep 1, 2025

@adamyeats adamyeats left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! One minor comment below.

Comment thread src/data/utils.ts Outdated
@adamyeats
adamyeats merged commit 952be45 into grafana:main Sep 4, 2025
20 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in Partner Datasources Sep 4, 2025
@alyssajoyner alyssajoyner mentioned this pull request Sep 30, 2025
@mcorbin
mcorbin deleted the fix-table-events-link branch October 17, 2025 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Span events & links not selected when querying trace from table view

4 participants