opentelemetry-proto icon indicating copy to clipboard operation
opentelemetry-proto copied to clipboard

Support grpc-retry-pushback-ms

Open DylanRussell opened this issue 10 months ago • 5 comments

Fixes https://github.com/open-telemetry/opentelemetry-specification/issues/4513

I've tried to change the wording such that it's acceptable for SDKs to use gRPC retry config, which only respects the grpc-retry-pushback-ms header to handle all backoff / retry logic.

The advantage of this is that SDKs don't need to implement / maintain any backoff/retry logic and can hand it off to the official gRPC library.

DylanRussell avatar May 28 '25 19:05 DylanRussell

@DylanRussell, given https://github.com/open-telemetry/opentelemetry-proto/pull/669 is merged I think you can update this PR :wink:

pellared avatar Jul 01 '25 15:07 pellared

Ack.. I've merged and updated the wording to be closer to what you recommended.. PTAL when you have a chance

DylanRussell avatar Jul 07 '25 20:07 DylanRussell

Linter passing now

DylanRussell avatar Jul 17 '25 14:07 DylanRussell

@open-telemetry/spec-sponsors, @open-telemetry/technical-committee, PTAL

pellared avatar Jul 30 '25 16:07 pellared

I think we need a description of how the new and old implementations should interoperate. Explain the operation in the following 3 scenarios:

  • Old client implementing the spec before this change connects to a new server implementing the spec after this change.
  • New client connects to old server.
  • New client connects to new server.

tigrannajaryan avatar Aug 20 '25 14:08 tigrannajaryan