Skip to content

Conformity of client interceptor(s) with the official grpc.*ClientInterceptor-interfaces #3877

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conformity of client interceptor(s) with the official grpc.*ClientInterceptor-interfaces #3877

wants to merge 2 commits into from

Conversation

juandelacruz-calvo
Copy link

@juandelacruz-calvo juandelacruz-calvo commented Oct 17, 2025 •
edited
Loading

Summary

This PR is a rebased version of PR #3384 that aims to update the OpenTelemetryClientInterceptor to support official gRPC client interceptor interfaces introduced in gRPC v1.8.0.

Changes

  • Remove external implementation of gRPC client interceptors (package grpcext)
  • Adapt the _client.py module based on the _aio_client.py module design
  • Remove the RpcInfo class from utilities.py
  • Update test_client_interceptor_trace_context_propagation tests
  • Update semantic conventions to use the new incubating attributes

Test plan

Tested locally, working with grpc client instrumentation, also working with PubSub consuming and publishing

Related Issues

🤖 Generated with Claude Code
@Sovietaced @juandelacruz-calvo
Conformity of client intereptor(s) with the official grpc.*ClientInte…
a4a8fc5 
…rceptor-interfaces Signed-off-by: Jason Parraga <sovietaced@gmail.com>
@Sovietaced @juandelacruz-calvo
Remove RpcInfo class
3327b42 
Signed-off-by: Jason Parraga <sovietaced@gmail.com>
@juandelacruz-calvo juandelacruz-calvo marked this pull request as ready for review October 17, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@juandelacruz-calvo @Sovietaced