Skip to content

Hybrid support for NRNS header for message queues and async kafka producers#2939

Open
jbedell-newrelic wants to merge 12 commits into
mainfrom
NewRelicNotSampled
Open

Hybrid support for NRNS header for message queues and async kafka producers#2939
jbedell-newrelic wants to merge 12 commits into
mainfrom
NewRelicNotSampled

Conversation

@jbedell-newrelic

@jbedell-newrelic jbedell-newrelic commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Updates to support the "nrns" header in message queues when the transaction is not selected for sampling.
Updates to support the otel autoconfigure use of span attributes for the same.
Removal of the Span ID value from the tracestate header as it is duplicated elsewhere.

Resolves: #2888
Resolves: #2952
Resolves: #2986

@codecov-commenter

codecov-commenter commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 15.78947% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.83%. Comparing base (edff1ac) to head (cf416ae).
⚠️ Report is 76 commits behind head on main.

Files with missing lines Patch % Lines
.../src/main/java/com/newrelic/agent/HeadersUtil.java 6.25% 12 Missing and 3 partials ⚠️
...ewrelic/agent/config/DistributedTracingConfig.java 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2939      +/-   ##
============================================
+ Coverage     70.78%   70.83%   +0.05%     
+ Complexity    10664    10642      -22     
============================================
  Files           879      870       -9     
  Lines         42971    42894      -77     
  Branches       6501     6489      -12     
============================================
- Hits          30419    30386      -33     
+ Misses         9623     9562      -61     
- Partials       2929     2946      +17     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jbedell-newrelic jbedell-newrelic moved this from Triage to Reviewed in Java Engineering Board Jun 16, 2026
@jbedell-newrelic
jbedell-newrelic marked this pull request as ready for review June 23, 2026 13:18
jtduffy
jtduffy previously approved these changes Jun 23, 2026
@jbedell-newrelic jbedell-newrelic changed the title First commit working version, cleanup still to do Updates to support the NRNS header for message queues Jun 29, 2026
jtduffy
jtduffy previously approved these changes Jun 29, 2026
@jbedell-newrelic jbedell-newrelic changed the title Updates to support the NRNS header for message queues Hybrid support for NRNS header for message queues and async kafka producers Jul 17, 2026
jtduffy
jtduffy previously approved these changes Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Reviewed

Development

Successfully merging this pull request may close these issues.

Hybrid agent missing async producer spans Deduplicate span id from headers Minimize DT headers when not sampled

3 participants