Make exporter `timeout` encompass retries/backoffs, add jitter to backoffs, cleanup code a bit by DylanRussell · Pull Request #4564 · open-telemetry/opentelemetry-python

added 2 commits

April 30, 2025 17:41

@DylanRussell

emdneto

@DylanRussell

@DylanRussell

@DylanRussell

@DylanRussell

emdneto

emdneto

@DylanRussell

@DylanRussell

aabmass

@DylanRussell

@DylanRussell

@DylanRussell DylanRussell changed the title Add a timeout param to all OTLP grpc / http export calls -- fixed merge conflicts Switch gRPC exporters to use official gRPC retry config. Make timeout encompass retries/backoffs

May 22, 2025

@DylanRussell

@DylanRussell

@DylanRussell

@DylanRussell

emdneto

@DylanRussell @emdneto

…try/exporter/otlp/proto/grpc/exporter.py

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>

emdneto

@DylanRussell

emdneto

@DylanRussell

@xrmx

xrmx

xrmx approved these changes Jun 13, 2025

@xrmx xrmx enabled auto-merge (squash)

June 13, 2025 07:29

@xrmx

mxiamxia pushed a commit to aws-observability/aws-otel-python-instrumentation that referenced this pull request

Oct 30, 2025
…58b0 (#524)

This PR updates the upstream OpenTelemetry Python dependency to its
September 2025 release; Upgrading from version 1.33.1/0.54b1 to
1.37.0/0.58b0.

It also resolves several conflicts between the following OTel PRs and
existing ADOT patches:

starlette: Remove maximum version constraint  

open-telemetry/opentelemetry-python-contrib#3456

Make a BatchProcessor class which both BatchSpanRecordProcessor and
BatchLogRecordProcessor can use
open-telemetry/opentelemetry-python#4562

Make exporter timeout encompass retries/backoffs, add jitter to
backoffs, cleanup code a bit
open-telemetry/opentelemetry-python#4564

Update BatchSpanProcessor to use new BatchProcessor class 
open-telemetry/opentelemetry-python#4580

Fix issue where deadlock can occur over logging._lock 
open-telemetry/opentelemetry-python#4636

Tests Performed

tox -e lint
tox -e spellcheck
tox -e 3.9-test-aws-opentelemetry-distro
tox -e 3.10-test-aws-opentelemetry-distro
tox -e 3.11-test-aws-opentelemetry-distro
tox -e 3.12-test-aws-opentelemetry-distro
tox -e 3.13-test-aws-opentelemetry-distro

Smoke/contract tests: ./gradlew
appsignals-tests:contract-tests:contractTests

By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.

JWinermaSplunk pushed a commit to JWinermaSplunk/opentelemetry-python that referenced this pull request

Feb 17, 2026
…koffs, cleanup code a bit (open-telemetry#4564)

* Initial commit to add timeout as a parm to export, make retries encompass timeout

* Fix lint issues

* Fix a bunch of failing style/lint/spellcheck checks

* Remove timeout param from the export calls.

* Fix flaky windows test ?

* Respond to review comments..

* Delete exponential backoff code that is now unused

* Add changelog and remove some unused imports..

* fix typo and unit test flaking on windows

* Refactor tests, HTTP exporters a bit

* Remove unneeded test reqs

* Remove gRPC retry config

* Tweak backoff calculation

* Lint and precommit

* Empty commit

* Another empty commit

* Calculate backoff in 1 place instead of 2

* Update changelog

* Update changelog

* Make new _common directory in the http exporter for shared code

* precommit

* Respond to comments on PR

* Fix broken test, execute precommit

* Skip some tests on windows

* Explain why test is skipped

* Update exporter/opentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/exporter.py

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>

* Revert change to start respecting timeout passed into metric exporter

---------

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>
Co-authored-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>

JWinermaSplunk pushed a commit to JWinermaSplunk/opentelemetry-python that referenced this pull request

Feb 17, 2026
…koffs, cleanup code a bit (open-telemetry#4564)

* Initial commit to add timeout as a parm to export, make retries encompass timeout

* Fix lint issues

* Fix a bunch of failing style/lint/spellcheck checks

* Remove timeout param from the export calls.

* Fix flaky windows test ?

* Respond to review comments..

* Delete exponential backoff code that is now unused

* Add changelog and remove some unused imports..

* fix typo and unit test flaking on windows

* Refactor tests, HTTP exporters a bit

* Remove unneeded test reqs

* Remove gRPC retry config

* Tweak backoff calculation

* Lint and precommit

* Empty commit

* Another empty commit

* Calculate backoff in 1 place instead of 2

* Update changelog

* Update changelog

* Make new _common directory in the http exporter for shared code

* precommit

* Respond to comments on PR

* Fix broken test, execute precommit

* Skip some tests on windows

* Explain why test is skipped

* Update exporter/opentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/exporter.py

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>

* Revert change to start respecting timeout passed into metric exporter

---------

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>
Co-authored-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>