Skip to content

Commit

Permalink
Do not purge all bank snapshots after fastboot (solana-labs#345)
Browse files Browse the repository at this point in the history
  • Loading branch information
brooksprumo authored Mar 28, 2024
1 parent b9a998a commit 8822aaa
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions ledger/src/bank_forks_utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -297,15 +297,6 @@ fn bank_forks_from_snapshot(
source: err,
path: fastboot_snapshot.snapshot_path(),
})?;

// If the node crashes before taking the next bank snapshot, the next startup will attempt
// to load from the same bank snapshot again. And if `shrink` has run, the account storage
// files that are hard linked in bank snapshot will be *different* than what the bank
// snapshot expects. This would cause the node to crash again. To prevent that, purge all
// the bank snapshots here. In the above scenario, this will cause the node to load from a
// snapshot archive next time, which is safe.
snapshot_utils::purge_all_bank_snapshots(&snapshot_config.bank_snapshots_dir);

bank
} else {
// Given that we are going to boot from an archive, the append vecs held in the snapshot dirs for fast-boot should
Expand Down

0 comments on commit 8822aaa

Please sign in to comment.