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

Commit

Permalink
FM-365: Comments about what we could do about the snapshots in tmp
Browse files Browse the repository at this point in the history
  • Loading branch information
aakoshh committed Nov 22, 2023
1 parent 82b193c commit ce7ffa8
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions fendermint/app/src/app.rs
Original file line number Diff line number Diff line change
Expand Up @@ -909,6 +909,15 @@ where
state.block_height = snapshot.manifest.block_height;
state.state_params = snapshot.manifest.state_params;
self.set_committed_state(state)?;

// TODO: We can remove the `current_download` from the STM
// state here which would cause it to get dropped from /tmp,
// but for now let's keep it just in case we need to investigate
// some problem.

// We could also move the files into our own snapshot directory
// so that we can offer it to others, but again let's hold on
// until we have done more robust validation.
}
return Ok(response::ApplySnapshotChunk {
result: response::ApplySnapshotChunkResult::Accept,
Expand Down

0 comments on commit ce7ffa8

Please sign in to comment.