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

Support for CNCF Jaeger #77

Closed
SergeyKanzhelev opened this issue Jun 3, 2019 · 1 comment
Closed

Support for CNCF Jaeger #77

SergeyKanzhelev opened this issue Jun 3, 2019 · 1 comment

Comments

@SergeyKanzhelev
Copy link
Member

Need to support Jaeger exporter
@SergeyKanzhelev SergeyKanzhelev mentioned this issue Jun 3, 2019
Closed
@SergeyKanzhelev SergeyKanzhelev added this to the Basic exporters and adapters milestone Jun 11, 2019
EGHornbostel added a commit to PriceSpider-NeuIntel/opentelemetry-dotnet that referenced this issue Jul 3, 2019
@EGHornbostel
Added CNCF Jaeger exporter support
6d37ed2 
open-telemetry#77
@EGHornbostel EGHornbostel mentioned this issue Jul 3, 2019
Merged
austinlparker pushed a commit that referenced this issue Jul 24, 2019
@EGHornbostel @austinlparker
Jaeger exporter implementation (#132)
8e1d8fa 
* Fix visual studio dotnet core build task

* Added CNCF Jaeger exporter support

#77

* possible fix for win32 build issue.

* Fix windows build by adding Thrift source

* Fix yet more windows build issues.

* fix another style issue.

* fix reference to System.Net.Http

* remove unnecessary conditional framework reference.

* Update lib/Thrift/Transports/Client/TStreamClientTransport.cs

Co-Authored-By: Bruno Garcia <[email protected]>

* Update lib/Thrift/Transports/Client/TStreamClientTransport.cs

Co-Authored-By: Bruno Garcia <[email protected]>

* Added the conditional for netstandard2.0

* remove IDisposable TODO comments.

* Update src/OpenTelemetry.Exporter.Jaeger/Implimentation/EmitBatchArgs.cs

Co-Authored-By: Bruno Garcia <[email protected]>

* Add readme with thrift fork info.

* cleanup after merge.

* WIP remove thrift code

* Use prepackaged Apache Thrift assembly

* wrap EmitBatch

* Revert "wrap EmitBatch "

This reverts commit 9aaa138.

* Revert "Use prepackaged Apache Thrift assembly"

This reverts commit fa5fbd8.

* Revert "Use prepackaged Apache Thrift assembly"

This reverts commit fa5fbd8.

* Revert "WIP remove thrift code"

This reverts commit bbbcf47.

* Add missing ConfigureAwait(false) to Thrift async calls.

* Added Thrift fork differences to README.md

* WIP Jaeger Unit tests

* Fix build after merge.

* WIP Jaeger Unit tests - Span conversion to Jaeger Thrift format

* WIP Jaeger Unit tests - Span conversion to Jaeger Thrift format - cleanup

* Clean up merge issues

* WIP

* WIP cleanup

* more testing WIP

* use pattern matching for attribute conversion

* WIP cleanup after timestamp removal.

* WIP

* WIP UDP integration test

* Update src/OpenTelemetry.Exporter.Jaeger/Implimentation/Batch.cs

Co-Authored-By: Bruno Garcia <[email protected]>

* remove solution-explorer

* add to gitignore

* refactored List<T> to IEnumerable<T>

* Fix unit test bug

* Add integration test

* cleanup

* fix unix epoch conversion

* Update Thrift expected payload in unit test.

* fix build issues

* fix typo

* fix solution
@bruno-garcia
Copy link
Member

I this this can be closed since #132 was merged.

Yun-Ting pushed a commit to Yun-Ting/opentelemetry-dotnet that referenced this issue Oct 13, 2022
@srprash
Fix test project name in AWS instrumentation (open-telemetry#77)
5b5719f 
Sorry I missed this one before merging in the whole PR. Doing the same fix as done for the rest of the workflows in open-telemetry/opentelemetry-dotnet-contrib#73
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bruno-garcia @austinlparker @SergeyKanzhelev