Skip to content

Remove the deprecated otelgrpc.StreamClientInterceptor#7646

Merged
pellared merged 10 commits intoopen-telemetry:mainfrom
MrAlias:rm-StreamClientInterceptor
Aug 4, 2025
Merged

Remove the deprecated otelgrpc.StreamClientInterceptor#7646
pellared merged 10 commits intoopen-telemetry:mainfrom
MrAlias:rm-StreamClientInterceptor

Conversation

@MrAlias
Copy link
Copy Markdown
Contributor

@MrAlias MrAlias commented Jul 31, 2025

Fix #7105

Indirect clean-up included:

  • There are no more tests in interceptor_test.go. The file is deleted, and all helper functions are moved to the unique locations they are now used.
  • Unused declarations were removed.

@MrAlias MrAlias added this to the v1.38.0 milestone Jul 31, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.7%. Comparing base (31a4771) to head (ca9495b).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #7646     +/-   ##
=======================================
- Coverage   78.7%   78.7%   -0.1%     
=======================================
  Files        186     186             
  Lines      14805   14679    -126     
=======================================
- Hits       11664   11556    -108     
+ Misses      2795    2782     -13     
+ Partials     346     341      -5     
Files with missing lines Coverage Δ
...ion/google.golang.org/grpc/otelgrpc/interceptor.go 65.0% <ø> (-15.2%) ⬇️

... and 1 file with indirect coverage changes

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

@MrAlias MrAlias force-pushed the rm-StreamClientInterceptor branch from be43406 to f05d456 Compare July 31, 2025 19:25
@MrAlias MrAlias marked this pull request as ready for review July 31, 2025 19:31
@MrAlias MrAlias requested review from a team and dashpole as code owners July 31, 2025 19:31
@pellared pellared merged commit 74313f7 into open-telemetry:main Aug 4, 2025
27 checks passed
@MrAlias MrAlias mentioned this pull request Aug 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove the deprecated StreamClientInterceptor function from otelgrpc

3 participants