Remove 'backoff' dependency and use a function instead by pmcollins · Pull Request #3679 · open-telemetry/opentelemetry-python
Navigation Menu
{{ message }}
open-telemetry / opentelemetry-python Public
- Notifications You must be signed in to change notification settings
- Fork 834
Merged
Remove 'backoff' dependency and use a function instead#3679
ocelotl merged 7 commits intoopen-telemetry:mainfrom
Remove 'backoff' dependency and use a function instead#3679
ocelotl merged 7 commits intoopen-telemetry:mainfrom
Conversation
Copy link Copy Markdown
Member
Description
The backoff dependency doesn't appear to have been needed as used, and it has caused at least one reported dependency conflict for an OTel Python user/developer. This change removes the dependency and replaces it with a function.
Fixes #3680
pmcollins
force-pushed
the
backoff
branch
3 times, most recently
from
7a0a059 to
7b48824
Compare
pmcollins
mentioned this pull request
pmcollins
force-pushed
the
backoff
branch
from
7b48824 to
4bb14ec
Compare
pmcollins added 2 commits
February 9, 2024 08:08
pmcollins
force-pushed
the
backoff
branch
from
4bb14ec to
d3abeb6
Compare
pmcollins
marked this pull request as ready for review
pmcollins
requested a review
from a team
sfc-gh-bdrutu approved these changes Feb 9, 2024
bogdandrutu approved these changes Feb 9, 2024
ocelotl suggested changes Feb 12, 2024
exporter/opentelemetry-exporter-otlp-proto-common/tests/test_backoff.py
Outdated
Show resolved
Hide resolved
exporter/opentelemetry-exporter-otlp-proto-common/tests/test_backoff.py Outdated Show resolved Hide resolved
ocelotl approved these changes Feb 12, 2024
ocelotl added 3 commits
February 13, 2024 10:03
ocelotl
enabled auto-merge (squash)
ocelotl
merged commit
20622fd
into
open-telemetry:main
lzchen
mentioned this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants