Skip to content

opentelemetry-exporter-otlp-proto-grpc: remove dependence on incubating rpc.response.status_code semconv attribute#5388

Open
herin049 wants to merge 2 commits into
open-telemetry:mainfrom
herin049:fix/grpc-incubating-semconv
Open

opentelemetry-exporter-otlp-proto-grpc: remove dependence on incubating rpc.response.status_code semconv attribute#5388
herin049 wants to merge 2 commits into
open-telemetry:mainfrom
herin049:fix/grpc-incubating-semconv

Conversation

@herin049

@herin049 herin049 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Description

Removes the dependence on the incubating rpc.response.status_code semconv attribute in the OTLP gRPC exporter.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

uv run tox -e py314-test-opentelemetry-exporter-otlp-proto-grpc-latest

Does This PR Require a Contrib Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added

…ng rpc.response.status_code semconv attribute
@herin049 herin049 requested a review from a team as a code owner July 4, 2026 02:32
@xrmx

xrmx commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

@herin049 Could you please elaborate on why we do want this?

@herin049

Copy link
Copy Markdown
Contributor Author

@herin049 Could you please elaborate on why we do want this?

opentelemetry.semconv._incubating.attributes.rpc_attributes is internal, so it is not expected to be stable across minor versions. However the gRPC exporter has the requirement opentelemetry-sdk ~= 1.44.0.dev which is a lie.

It's the same argument for not importing internal methods from the SDK into exporter packages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants