-
Notifications
You must be signed in to change notification settings - Fork 11.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tempo: Embed flame graph in span details #77537
Conversation
.../app/features/explore/TraceView/components/TraceTimelineViewer/SpanDetail/KeyValuesTable.tsx
Outdated
Show resolved
Hide resolved
public/app/features/explore/TraceView/components/TraceTimelineViewer/SpanDetail/index.tsx
Outdated
Show resolved
Hide resolved
.../app/features/explore/TraceView/components/TraceTimelineViewer/SpanDetail/KeyValuesTable.tsx
Outdated
Show resolved
Hide resolved
public/app/features/explore/TraceView/components/TraceTimelineViewer/SpanDetail/index.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Explore.tsx LGTM 👍
I think this looks good but I would like to merge this behind different feature flag than the So I think it would be better to have another feature flag so we don't have to wait with the linking itself, while we can merge this and keep iterating on this. |
* Embed flame graph * Update test * Update test * Use toggle * Update test * Add tests * Use const * Cleanup * Update profile tag * Move flame graph out of tags, remove request and other cleanup + tests * Update test * Set flame graph by profile id and simplify logic * Cleanup and redrawListView * Create/use feature toggle
What is this feature?
Embeds a flame graph in Tempo's span details (if one exists).
Why do we need this feature?
So users can visualise profiles in their span details.
Who is this feature for?
Tempo/Pyroscope users.
Special notes for your reviewer:
You can test with the Pyroscope docker-compose.
Need to enable the feature toggle
tracesEmbeddedFlameGraph
.Please check that: