Skip to content
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

Tracing: Trace to profiles #76670

Merged
merged 41 commits into from
Nov 1, 2023
Merged

Tracing: Trace to profiles #76670

merged 41 commits into from
Nov 1, 2023

Conversation

joey-grafana
Copy link
Contributor

@joey-grafana joey-grafana commented Oct 17, 2023

What is this feature?

Trace to profiles support.

Why do we need this feature?

Allows users to easily link from their tracing to their profiling data source.

Who is this feature for?

Tracing/profiling users.

Special notes for your reviewer:

You can test with the Pyroscope docker-compose.
Need to enable the feature toggle traceToProfiles.

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@aocenas
Copy link
Member

aocenas commented Oct 27, 2023

Trying this with the pyro test docker compose but I don't see any link.

Screenshot 2023-10-27 at 17 17 25

Is it possible the pyroscope.profile.id isn't added anymore?

@joey-grafana
Copy link
Contributor Author

@aocenas yeah this was updated in the time since I created the PR. Updated that now so all good for review.

@joey-grafana joey-grafana merged commit c39e9a8 into main Nov 1, 2023
21 of 22 checks passed
@joey-grafana joey-grafana deleted the joey/trace-to-profiles branch November 1, 2023 10:14
@aangelisc aangelisc modified the milestones: 10.3.x, 10.2.3 Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants