Skip to content

otlp: Respect backoff in retry time limit - #8845

Open
Sakshamm-Goyal wants to merge 2 commits into
open-telemetry:mainfrom
Sakshamm-Goyal:fix/otlp-retry-elapsed-backoff
Open

otlp: Respect backoff in retry time limit#8845
Sakshamm-Goyal wants to merge 2 commits into
open-telemetry:mainfrom
Sakshamm-Goyal:fix/otlp-retry-elapsed-backoff

Conversation

@Sakshamm-Goyal

Copy link
Copy Markdown

Fixes #8781.

The retry loop already waits for the greater of the exponential backoff and server throttle, but its remaining-budget check only included the throttle. When backoff was greater, an exporter could wait past MaxElapsedTime and issue another request.

This changes the budget check to use the selected delay and adds the regression to the shared retry template, regenerating all six OTLP trace, metric, and log HTTP/gRPC implementations.

Validation:

  • reproduced the regression before the fix in the generated trace HTTP retry package
  • exact regression passes in all six generated retry packages
  • full retry suites pass in all six generated packages
  • make precommit passes, including generation, tidy, lint, README/module verification, and repository-wide race tests
  • git diff --check passes

@linux-foundation-easycla

linux-foundation-easycla Bot commented Aug 23, 2026

Copy link
Copy Markdown

CLA Not Signed

@MrAlias

MrAlias commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

@Sakshamm-Goyal please sign the CLA so these changes can be considered.

@MrAlias MrAlias added the blocked:CLA Waiting on CLA to be signed before progress can be made label Aug 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked:CLA Waiting on CLA to be signed before progress can be made

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OTLP retry MaxElapsedTime check ignores the selected backoff delay

2 participants