-
Notifications
You must be signed in to change notification settings - Fork 44
Fix: include the two latest ledger state snapshots in the ancillary archive #2484
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
Fix: include the two latest ledger state snapshots in the ancillary archive #2484
Conversation
Test Results 3 files ±0 57 suites ±0 11m 7s ⏱️ -26s Results for commit 8e13f58. ± Comparison against base commit 4d382b9. This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
mithril-aggregator/src/services/snapshotter/compressed_archive_snapshotter.rs
Outdated
Show resolved
Hide resolved
4f652a9
to
21d837e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
mithril-aggregator/src/services/snapshotter/compressed_archive_snapshotter.rs
Outdated
Show resolved
Hide resolved
… ancillary archive
* mithril-aggregator from `0.7.51` to `0.7.52`
21d837e
to
8e13f58
Compare
Content
This PR includes a fix that adds the two most recent ledger state snapshots to the ancillary archive.
It improves the reliability of Cardano node fast bootstrap, preventing full chain replays when restarting the node.
Pre-submit checklist
Issue(s)
Closes #2474