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

feat: add new internal exponential "backoff" decorator util #435

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

a-dubs
Copy link
Collaborator

@a-dubs a-dubs commented Oct 1, 2024

By implementing this as internal util, we avoid bringing in another dependency to add such simple functionality. Thorough unit tests were added to ensure this backoff decorator functions as desired.

@a-dubs a-dubs force-pushed the add-custom-backoff branch 4 times, most recently from 45a759d to 91a3a6d Compare October 2, 2024 03:36
@aciba90 aciba90 self-assigned this Oct 2, 2024
By implementing this as internal util, we avoid bringing in another
dependency to add such simple functionality. Thorough unit tests were
added to ensure this backoff decorator functions as desired.
This commit adds logic so that Quota related errors can be retried on VM creation. Adding this helps facilitate clearing of dangling resources on VM creation.
@r3tro7
Copy link

r3tro7 commented Oct 4, 2024

Thank you for the great work on this @a-dubs !
I've reviewed the changes and everything looks good and seems to work as intended.

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

Successfully merging this pull request may close these issues.

3 participants