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

config: update MaxCatchpointDownloadDuration to 12h #5503

Merged
merged 1 commit into from Jun 27, 2023

Conversation

cce
Copy link
Contributor

@cce cce commented Jun 27, 2023

Summary

Today the catchpoint restore system waits for a maximum of 2 hours to download a snapshot before giving up. Since this parameter was originally set, the size of a snapshot has increased by several times. This increases the default MaxCatchpointDownloadDuration configuration value to 12h.

Test Plan

Configuration update only; existing tests should pass.

@cce cce changed the title update MaxCatchpointDownloadDuration to 12h config: update MaxCatchpointDownloadDuration to 12h Jun 27, 2023
@codecov
Copy link

codecov bot commented Jun 27, 2023

Codecov Report

Merging #5503 (161d212) into master (df22f75) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5503      +/-   ##
==========================================
- Coverage   55.82%   55.80%   -0.02%     
==========================================
  Files         446      446              
  Lines       63224    63224              
==========================================
- Hits        35292    35283       -9     
- Misses      25565    25569       +4     
- Partials     2367     2372       +5     
Impacted Files Coverage Δ
config/localTemplate.go 70.76% <ø> (ø)

... and 14 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cce cce merged commit 068b375 into algorand:master Jun 27, 2023
17 of 21 checks passed
@cce cce deleted the MaxCatchpointDownloadDuration-12h branch June 27, 2023 15:28
ASISBusiness

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants