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.

Sign up for GitHub

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

Jump to bottom

[Incremental Snapshots] Add integration tests #18829

Closed
Tracked by #17088
brooksprumo opened this issue Jul 21, 2021 · 1 comment · Fixed by #18565
Closed
Tracked by #17088

[Incremental Snapshots] Add integration tests #18829

brooksprumo opened this issue Jul 21, 2021 · 1 comment · Fixed by #18565
Assignees
@brooksprumo
Labels
locked issue

Comments

@brooksprumo
Copy link
Contributor

Just like the title says. Created this issue so its easier to track in the larger #17088 issue.

Add integration tests for incremental snapshots to core/tests/snapshots.rs.
@brooksprumo brooksprumo self-assigned this Jul 21, 2021
@brooksprumo brooksprumo mentioned this issue Jul 21, 2021
Closed
29 tasks
@brooksprumo brooksprumo linked a pull request Jul 21, 2021 that will close this issue
Add test_bank_forks_incremental_snapshot() #18565
Merged
brooksprumo added a commit that referenced this issue Jul 29, 2021
@brooksprumo
Add test_bank_forks_incremental_snapshot() (#18565)
b05fb87 
This commit builds on PR #18504 by adding a test to core/tests/snapshot.rs for Incremental Snapshots. The test adds banks to bank forks in a loop and takes both full snapshots and incremental snapshots at intervals, and validates they are rebuild-able.

For background info about Incremental Snapshots, see #17088.

Fixes #18829 and #18972
@github-actions
Copy link
Contributor

github-actions bot commented Apr 2, 2022

This issue has been automatically locked since there has not been any activity in past 7 days after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@brooksprumo brooksprumo

Labels
locked issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add test_bank_forks_incremental_snapshot() brooksprumo/solana
1 participant
@brooksprumo