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

Decrease retry intervals for provider uptime checks #288

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

Redm4x
Copy link
Contributor

@Redm4x Redm4x commented Aug 5, 2024

Decrease retry intervals for provider uptime checks (#286)

This PR adjust the retry intervals to be more frequent when a provider goes offline. The goal is to detect earlier when a provider comes back online. The retry frequency is also capped to 5 min for new providers (< 7d) to give them more time to finish setting up their provider before the interval starts to increase.

New Retry Intervals

Downtime Duration Retry Frequency
< 30m 1m
30m - 6h 5m*
6h - 24h 15m
24h - 7d 30m
7d - 30d 1h
30d+ 24h

* 5min is the max retry interval for new providers (< 7d)

Old Retry Intervals (for comparison)

Downtime Duration Retry Frequency
< 15m 1m
15m - 1h 5m
1h - 6h 15m
6h - 24h 30m
24h - 7d 1h
7d+ 24h

@anilmurty
Copy link

Sounds good to me - thanks @Redm4x (cc @andy108369 and @devalpatel67 in case they have any thoughts/ feedback)

@anilmurty
Copy link

oh and I did a search of docs to see whether anything needs to be updated and did not find anything -- not sure if you are aware of any @chainzero ?

@devalpatel67
Copy link
Contributor

Looks good to me.

@Redm4x Redm4x merged commit b883c80 into main Aug 12, 2024
5 checks passed
@Redm4x Redm4x deleted the fix/reduce-retry-intervals branch August 12, 2024 18:54
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.

4 participants