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

[CN-3737] increase wait time for application creation #9

Merged
merged 1 commit into from
Dec 3, 2024
Merged

Conversation

arodski
Copy link

@arodski arodski commented Dec 3, 2024

Summary

This change updates the polling mechanism to use exponential backoff with a quadratic increase in wait time between API calls, for a maximum of 20 minutes. The sleep duration is capped at 30 seconds to prevent excessive delays.

Example

For successive attempts:

Attempt 1: Sleep 1 second.
Attempt 2: Sleep 4 seconds.
Attempt 3: Sleep 9 seconds.
Attempt 6+: Sleep capped at 30 seconds.

Testing

Error returned after max attempts reached

fail

Successful after 10 minutes

success

@arodski arodski force-pushed the CN-3737 branch 2 times, most recently from 5e31a80 to 03173d1 Compare December 3, 2024 18:31
Copy link

@guido9j guido9j left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@arodski arodski merged commit 007ed53 into master Dec 3, 2024
3 checks passed
@arodski arodski deleted the CN-3737 branch December 3, 2024 19:58
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.

2 participants