Skip to content

Cleanup managed channel instances in jaeger remote sampler tests#8183

Merged
jack-berg merged 1 commit intoopen-telemetry:mainfrom
jack-berg:jaeger-sampler-build-noise
Mar 12, 2026
Merged

Cleanup managed channel instances in jaeger remote sampler tests#8183
jack-berg merged 1 commit intoopen-telemetry:mainfrom
jack-berg:jaeger-sampler-build-noise

Conversation

@jack-berg
Copy link
Copy Markdown
Member

@jack-berg jack-berg commented Mar 12, 2026

Removes build output like:

[Test worker] ERROR io.grpc.internal.ManagedChannelOrphanWrapper - *~*~*~ Previous channel ManagedChannelImpl{logId=5, target=127.0.0.1:57230} was garbage collected without
  being shut down! ~*~*~*
          Make sure to call shutdown()/shutdownNow()
      java.lang.RuntimeException: ManagedChannel allocation site

@jack-berg jack-berg requested a review from a team as a code owner March 12, 2026 17:42
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.30%. Comparing base (839f235) to head (6dd0fd0).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #8183   +/-   ##
=========================================
  Coverage     90.29%   90.30%           
- Complexity     7650     7651    +1     
=========================================
  Files           843      843           
  Lines         23059    23061    +2     
  Branches       2309     2310    +1     
=========================================
+ Hits          20822    20826    +4     
+ Misses         1519     1516    -3     
- Partials        718      719    +1     

☔ View full report in Codecov by Sentry.
📢 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.

@jack-berg jack-berg merged commit 777213e into open-telemetry:main Mar 12, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants