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

OTel span support #1886

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

OTel span support #1886

wants to merge 15 commits into from

Conversation

sdaubin
Copy link
Collaborator

@sdaubin sdaubin commented May 7, 2024

Overview

The java agent supports auto configuring the OTel SDK to report data to New Relic. Spans that are created with the span api are orphaned from New Relic data structures though. This returns spans backed by NR traces to generate metric and DT data.

This change hooks the SdkTracerProvider.tracerBuilder method and returns our tracer builder. When spans are started, we attempt to create a tracer. If successful, we return a Span implementation backed by the tracer, otherwise we return a no op span.

There is some special logic based on the span kind:

consumer

We start an OtherTransaction for consumer span kind.

server

We start a transaction for server spans and turn it into a web transaction.

client

For client spans, we either turn it into a database span or an external based on the span attributes.

Configuration

opentelemetry.sdk.spans.enabled can be use to disable the span behavior.
opentelemetry.instrumentation.{instrumentation-scope}.enabled can be used to disable spans for a specific Opentelemetry library

Related Github Issue

Include a link to the related GitHub issue, if applicable

Testing

The agent includes a suite of tests which should be used to
verify your changes don't break existing functionality. These tests will run with
Github Actions when a pull request is made. More details on running the tests locally can be found
here,

Checks

  • Your contributions are backwards compatible with relevant frameworks and APIs.
  • Your code does not contain any breaking changes. Otherwise please describe.
  • Your code does not introduce any new dependencies. Otherwise please describe.

@@ -59,6 +58,10 @@ ExitTracer createTracer(Object invocationTarget, int signatureId, boolean dispat

ExitTracer createScalaTxnTracer();

default ExitTracer createTracer(String metricName, int flags) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The createSegment API still generates an asynchronous tracer, which is not what we want. And from the bridge there's no easy way to register a signature to get a signatureId. This was the easiest way to get a normal, synchronous tracer.

@codecov-commenter
Copy link

codecov-commenter commented May 8, 2024

Codecov Report

Attention: Patch coverage is 35.71429% with 27 lines in your changes are missing coverage. Please review.

Project coverage is 70.63%. Comparing base (b582130) to head (4911fc1).
Report is 29 commits behind head on main.

Files Patch % Lines
...java/com/newrelic/agent/tracers/DefaultTracer.java 69.23% 1 Missing and 3 partials ⚠️
...ain/java/com/newrelic/agent/bridge/ExitTracer.java 0.00% 3 Missing ⚠️
...elic/agent/bridge/datastore/SqlQueryConverter.java 0.00% 3 Missing ⚠️
...ava/com/newrelic/agent/database/SqlObfuscator.java 25.00% 3 Missing ⚠️
...elic/agent/service/analytics/SpanEventFactory.java 50.00% 2 Missing and 1 partial ⚠️
...ava/com/newrelic/agent/tracers/AbstractTracer.java 0.00% 3 Missing ⚠️
...n/java/com/newrelic/agent/bridge/TracedMethod.java 0.00% 2 Missing ⚠️
...lic/agent/instrumentation/InstrumentationImpl.java 0.00% 2 Missing ⚠️
...gent/instrumentation/pointcuts/XmlRpcPointCut.java 0.00% 2 Missing ⚠️
...ava/com/newrelic/agent/bridge/Instrumentation.java 0.00% 1 Missing ⚠️
... and 1 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1886      +/-   ##
============================================
- Coverage     70.66%   70.63%   -0.04%     
+ Complexity     9876     9861      -15     
============================================
  Files           828      830       +2     
  Lines         39848    39841       -7     
  Branches       6064     6066       +2     
============================================
- Hits          28160    28141      -19     
- Misses         8960     8970      +10     
- Partials       2728     2730       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sdaubin sdaubin requested a review from jasonjkeller May 8, 2024 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Needs Review
Development

Successfully merging this pull request may close these issues.

None yet

2 participants