feat: make retryable gRPC error codes configurable for gRPC exporters#4917
feat: make retryable gRPC error codes configurable for gRPC exporters#4917herin049 wants to merge 9 commits intoopen-telemetry:mainfrom
Conversation
|
Would it be feasible to also support this config by env var? Maybe something like |
Definitely, I can add this! |
60c3e69 to
12ea823
Compare
opentelemetry-sdk/src/opentelemetry/sdk/environment_variables/__init__.py
Outdated
Show resolved
Hide resolved
tammy-baylis-swi
left a comment
There was a problem hiding this comment.
Overall lgtm as a nice, simple, experimental feature.
exporter/opentelemetry-exporter-otlp-proto-grpc/tests/test_otlp_exporter_mixin.py
Show resolved
Hide resolved
|
Can someone please have a look on the CI? The feature is really needed by me :-) |
7439e6a to
fa3f686
Compare
@kasium Just waiting for a maintainer to merge this in, should be merged shortly. |
fa3f686 to
f0df87e
Compare
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.
How Has This Been Tested?
Relevant unit tests have been added.
Does This PR Require a Contrib Repo Change?
Checklist: