Skip to content

Updates to support the NRNS header for message queues#2939

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

Updates to support the NRNS header for message queues#2939
jbedell-newrelic wants to merge 10 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

@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.75%. Comparing base (edff1ac) to head (68eb8de).
⚠️ Report is 50 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.75%   -0.04%     
+ Complexity    10664    10659       -5     
============================================
  Files           879      879              
  Lines         42971    42986      +15     
  Branches       6501     6504       +3     
============================================
- Hits          30419    30414       -5     
- Misses         9623     9639      +16     
- Partials       2929     2933       +4     

☔ 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
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.

Deduplicate span id from headers Minimize DT headers when not sampled

3 participants