Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ContainerMetadataCredentials should retry with backoff #193

Open
bennettgoble opened this issue Sep 30, 2024 · 0 comments
Open

ContainerMetadataCredentials should retry with backoff #193

bennettgoble opened this issue Sep 30, 2024 · 0 comments

Comments

@bennettgoble
Copy link

When running aiodynamo in an ECS application under high load it is possible to hit timeout issues accessing the container metadata endpoint. I believe the chance of re-connection could be improved if the retry logic implemented some basic backoff.

Traceback (most recent call last):
  File "uvloop/cbhandles.pyx", line 63, in uvloop.loop.Handle._run
  File "/opt/venv/lib/python3.12/site-packages/aiodynamo/credentials.py", line 289, in _set_current
    self._current = task.result()
                    ^^^^^^^^^^^^^
  File "/opt/venv/lib/python3.12/site-packages/aiodynamo/credentials.py", line 389, in fetch_metadata
    response = await fetch_with_retry_and_timeout(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/venv/lib/python3.12/site-packages/aiodynamo/credentials.py", line 604, in fetch_with_retry_and_timeout
    raise TooManyRetries()
aiodynamo.credentials.TooManyRetries
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant