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

Add OpenTelemetry Tracing #81

Closed
wants to merge 28 commits into from
Closed

Add OpenTelemetry Tracing #81

wants to merge 28 commits into from

Conversation

toizo
Copy link
Collaborator

@toizo toizo commented Mar 8, 2024 •
edited
Loading

Adds OpenTelemetry Tracing as discsussed in #50.

Includes:

  • refactored the dead branch (v0.12.0-otel1) and aligned it with newest OpenTelemetry semantic conventions.
  • The tracing context between consumer and producer propagates via a new traceparent message header on the envelope. This is aligned with a W3C Trace Context spec.
  • some minor linting (e.g. removing references of unused namespaces)

thofisch and others added 28 commits April 22, 2022 10:55
@thofisch @thfis-dfds
add open telemetry to producer
384139d
@thofisch @thfis-dfds
add open telemetry to consumer
4921d09
@thofisch @thfis-dfds
add open telemetry jaeger exporter to Tester project
d49b165
@thfis-dfds
add OTel messaging tags
774da91
@thfis-dfds
revert to block-scoped namespace
e2dc28f
@thfis-dfds
allow for pre-releases
16ea066
@thfis-dfds
extend logging around otel logging
77cdc39
@thfis-dfds
update version to 0.12.0-otel.1
d96ce80
@toizo
Merge branch 'master' into feature/otel
1570b44
@toizo
update version
62d07d9
@toizo
update messaging attributes according to the newest documentation
af759e1
@toizo
add OpenTelemetryMessagingAttributes
27e8b6b
@toizo
remove producing instrumentation temporarily
6299bd0
@toizo
move tracing from Consumer to ConsumerActivitySource
9548669
@toizo
remove logging from consumer
cbc7376
@toizo
move otel extensions to separate file
6a57d9a
@toizo
PayloadDescriptor: enable adding otel headers
566e709
@toizo
start the producing activity in producer instead of the inner producer
c897556
@toizo
linting
43343e7
@toizo
remove activity propagation from serializer
307699b
@toizo
remove unused namespaces
94f29c1
@toizo
rename test to fit its purpose
46c2e65
@toizo
diagnostics constants: align maming convention
ef5ecb0
@toizo
move ActivitySources to their own files
26b6515
@toizo
remove unused namespaces
5f25689
@toizo
linting
4f73e8c
@toizo
update OpenTelemetry.Api to v1.7.0
74cc3a2
@toizo
update tester project dependencies
3669f34
@toizo
Copy link
Collaborator Author

toizo commented Mar 13, 2024

I will be abandoning this PR since I prepared another one with a cleaner commit history

@toizo toizo closed this Mar 13, 2024
@toizo toizo deleted the feature/open-telemetry branch September 24, 2024 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martinosk martinosk Awaiting requested review from martinosk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@toizo @thofisch @thfis-dfds