-
-
Notifications
You must be signed in to change notification settings - Fork 235
feat: Add support to send OTEL traces via OTLP #4899
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
Merged
Merged
Changes from 24 commits
Commits
Show all changes
61 commits
Select commit
Hold shift + click to select a range
2122abb
Added support to send OTEL traces via OTLP
jamescrosswell a2fb0e4
Added OTEL propagation context to link OTEL spans to events etc.
jamescrosswell 442fc7d
Disable Sentry tracing instrumentation if OTEL is enabled
jamescrosswell 08d8711
Revert to classic extension methods
jamescrosswell 907850b
Added option to disable sentry tracing
jamescrosswell 6f8312d
Added some new tests... found some new issues
jamescrosswell 8fbbf89
Merge remote-tracking branch 'origin/main' into oltp-integration
jamescrosswell 1ea156a
Prevent Hub.GetBaggage from throwing an exception
jamescrosswell 83f9097
Fix tests
jamescrosswell ee67a6e
Bump to Otel 1.15.0 to address IL3050 error for AOT compilation
jamescrosswell d1597d8
changelog
jamescrosswell c8856a6
Fixed package downgrades
jamescrosswell 5ee6455
Fixed failing tests on .NET Framework
jamescrosswell 46eb4d6
Merge branch 'main' into oltp-integration
jamescrosswell af2457b
Format code
getsentry-bot ed9e04d
Moved to a separate NuGet package
jamescrosswell 304c5bc
.
jamescrosswell 4537ac9
.
jamescrosswell d2406ef
Updated Samples
jamescrosswell f1550ca
Review feedback
jamescrosswell 467b107
Consistently long argument names
jamescrosswell 918d400
Renamed AddSentryOtlp to AddSentryOtlpExporter
jamescrosswell db181ad
Removed unused code
jamescrosswell c3d8cc7
Apply suggestions from code review
jamescrosswell 1dcd704
Typos and redundant usings
jamescrosswell 92b2e0f
Merge remote-tracking branch 'origin/main' into oltp-integration
jamescrosswell 11831c1
Revoved manual changelog entry
jamescrosswell 758f2a1
Fix build error
jamescrosswell ba1489c
Revert unnecessary version bumps
jamescrosswell 7967973
Address naming conflicts
jamescrosswell 861ee93
Tests for UseOtlp
jamescrosswell fa48e02
Renamed package Sentry.OpenTelemetry.Exporter
jamescrosswell f1a6617
IsAotCompatible unconditional
jamescrosswell b6819b7
Remove workaround for Roslyn analyser bug (resolved)
jamescrosswell d35af03
Drop minimum OTEL version from 1.10.0 to 10.6.0
jamescrosswell f55ded4
Revert "Drop minimum OTEL version from 1.10.0 to 10.6.0"
jamescrosswell aff8691
Added IExternalPropagationContext
jamescrosswell 9789b53
Redundant comment
jamescrosswell 3130084
Simplified package reference
jamescrosswell 154d394
Format code
getsentry-bot 74cd8c0
Review feedback
jamescrosswell f83616f
Add custom collector url to AddSentryOtlpExporter
jamescrosswell f40de80
Tidy up
jamescrosswell d321b3f
Format code
getsentry-bot 26ca204
Tests for ActivityExtensions
jamescrosswell 6b60977
Tests for collectorUrl
jamescrosswell 9fef303
New hub tests
jamescrosswell db9885f
Parse sample rand and sample rate from OTEL TraceStateString
jamescrosswell 114127f
Format code
getsentry-bot acf14a4
Removed redundant guard
jamescrosswell dc518b5
Fixed inversions
jamescrosswell dad9e18
Handle possible whitespace in TraceStateString
jamescrosswell df952a0
Add additional headers to OTLP client requests
jamescrosswell f06a9ee
Deal with edge case on the rv being 0
jamescrosswell 534a7b7
Apply suggestions from code review
jamescrosswell dde314d
Merge remote-tracking branch 'origin/main' into oltp-integration
jamescrosswell e35fe31
perf: use ReadOnlySpan<char> in OtelPropagationContext to avoid heap …
jamescrosswell 45d8082
Fix race on DSC
jamescrosswell 17e5e64
Merge remote-tracking branch 'origin/oltp-integration' into oltp-inte…
jamescrosswell ddfb765
fix: snapshot Activity.Current in OtelPropagationContext to prevent T…
jamescrosswell e057d69
fix: fall through to scope PropagationContext in GetBaggage when no a…
jamescrosswell File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
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.
todo: @Flash0ver check release-registry (after merge to main && before releasing vNext)