Skip to content

feat: make retryable gRPC error codes configurable for gRPC exporters#4917

Open
herin049 wants to merge 8 commits intoopen-telemetry:mainfrom
herin049:feat/conf-grpc-retry-codes
Open

feat: make retryable gRPC error codes configurable for gRPC exporters#4917
herin049 wants to merge 8 commits intoopen-telemetry:mainfrom
herin049:feat/conf-grpc-retry-codes

Conversation

@herin049
Copy link

Description

Add support for configuring the retry-able gRPC error codes for gRPC exporters.

Fixes #4904

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?

Relevant unit tests have been added.

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

@herin049 herin049 requested a review from a team as a code owner February 17, 2026 05:19
@tammy-baylis-swi
Copy link
Contributor

Would it be feasible to also support this config by env var? Maybe something like OTEL_PYTHON_EXPORTER_OTLP_GRPC_RETRYABLE_ERROR_CODES, which is quite long but similar to other grpc exporter env vars.

@herin049
Copy link
Author

Would it be feasible to also support this config by env var? Maybe something like OTEL_PYTHON_EXPORTER_OTLP_GRPC_RETRYABLE_ERROR_CODES, which is quite long but similar to other grpc exporter env vars.

Definitely, I can add this!

@herin049 herin049 force-pushed the feat/conf-grpc-retry-codes branch from 0c79f4b to 60c3e69 Compare February 18, 2026 20:44
@herin049 herin049 force-pushed the feat/conf-grpc-retry-codes branch from 60c3e69 to 12ea823 Compare February 18, 2026 20:48
Copy link
Contributor

@tammy-baylis-swi tammy-baylis-swi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall lgtm as a nice, simple, experimental feature.

Copy link
Member

@MikeGoldsmith MikeGoldsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍🏻

"OTEL_PYTHON_EXPORTER_OTLP_GRPC_RETRYABLE_ERROR_CODES": ",INTERNAL, unknown,,,dEAdline_Exceeded "
},
)
def test_retryable_error_codes_initialization_from_env(self):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
def test_retryable_error_codes_initialization_from_env(self):
def test_retryable_error_codes_initialization_from_env(self):

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

Labels

None yet

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

Allow customization of retry error codes in OTLP grpc exporter

3 participants

Comments