Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Makes snapshot_utils aware of Incremental Accounts Hash #30804

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

brooksprumo
Copy link
Contributor

@brooksprumo brooksprumo commented Mar 20, 2023

Problem

snapshot_utils does not know about Incremental Accounts Hash, so the helps functions that create snapshots, and the functions that reconstruct a bank from snapshot archives cannot handle IAH.

Summary of Changes

snapshot_utils can now to do bank-to-snapshot-archive and bank-from-snapshot-archive for incremental snapshots with IAH.

Feature Gate Issue: #30819

@brooksprumo brooksprumo added work in progress This isn't quite right yet noCI Suppress CI on this Pull Request labels Mar 20, 2023
@brooksprumo brooksprumo self-assigned this Mar 20, 2023
@brooksprumo brooksprumo added CI Pull Request is ready to enter CI and removed noCI Suppress CI on this Pull Request labels Mar 21, 2023
@solana-grimes solana-grimes removed the CI Pull Request is ready to enter CI label Mar 21, 2023
@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #30804 (bc2da66) into master (bc933c6) will increase coverage by 0.0%.
The diff coverage is 98.5%.

@@           Coverage Diff            @@
##           master   #30804    +/-   ##
========================================
  Coverage    81.4%    81.4%            
========================================
  Files         725      725            
  Lines      203596   203721   +125     
========================================
+ Hits       165833   166030   +197     
+ Misses      37763    37691    -72     

@brooksprumo brooksprumo added the feature-gate Pull Request adds or modifies a runtime feature gate label Mar 21, 2023
@brooksprumo brooksprumo removed the work in progress This isn't quite right yet label Mar 21, 2023
@brooksprumo brooksprumo marked this pull request as ready for review March 21, 2023 16:13
Copy link
Contributor

@jeffwashington jeffwashington left a comment

Choose a reason for hiding this comment

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

lgtm

@brooksprumo brooksprumo added the automerge Merge this Pull Request automatically once CI passes label Mar 21, 2023
@mergify mergify bot merged commit b64d0de into solana-labs:master Mar 21, 2023
@brooksprumo brooksprumo deleted the iah/snapshot_utils branch March 21, 2023 16:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge Merge this Pull Request automatically once CI passes feature-gate Pull Request adds or modifies a runtime feature gate
Projects
Development

Successfully merging this pull request may close these issues.

3 participants