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.

Sign up for GitHub

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

Jump to bottom

fix(spans): Propagate profiler_id to all spans #3784

Merged
merged 4 commits into from
Jul 3, 2024
Merged

fix(spans): Propagate profiler_id to all spans #3784

merged 4 commits into from
Jul 3, 2024

Conversation

phacops
Copy link
Contributor

@phacops phacops commented Jul 2, 2024

Now that we ingest profiler_id in the context properly, we can propagate it to all spans so it gets ingested as part of the sentry_tags.

@phacops phacops requested a review from a team as a code owner July 2, 2024 21:09
@phacops phacops self-assigned this Jul 2, 2024
@phacops phacops merged commit e59b49b into master Jul 3, 2024
23 checks passed
@phacops phacops deleted the pierre/spans-tag-profiler_id-on-spans branch July 3, 2024 14:54
0Calories pushed a commit that referenced this pull request Jul 11, 2024
@phacops @0Calories
fix(spans): Propagate profiler_id to all spans (#3784)
be7783d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@viglia viglia viglia approved these changes

@Dav1dde Dav1dde Dav1dde approved these changes

Assignees

@phacops phacops

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@phacops @Dav1dde @viglia