Add sanity checks for weird SnapshotPolicyArgs configurations#1903
Open
fraser-iohk wants to merge 1 commit intomainfrom
Open
Add sanity checks for weird SnapshotPolicyArgs configurations#1903fraser-iohk wants to merge 1 commit intomainfrom
fraser-iohk wants to merge 1 commit intomainfrom
Conversation
fraser-iohk
requested review from
bladyjoker,
dnadales,
geo2a,
jasagredo and
nfrisby
as code owners
geo2a
force-pushed
the
amesgen/predictable-snapshots
branch
6 times, most recently
from
6258cef to
74d1d62
Compare
geo2a
force-pushed
the
amesgen/predictable-snapshots
branch
7 times, most recently
from
c6d09eb to
b1e2b2b
Compare
geo2a
force-pushed
the
fraser-iohk/snapshot-config-sanity-checks
branch
from
671f2d9 to
b2395be
Compare
geo2a
force-pushed
the
amesgen/predictable-snapshots
branch
8 times, most recently
from
2f522ff to
3c557b3
Compare
geo2a
force-pushed
the
amesgen/predictable-snapshots
branch
from
3c557b3 to
f896763
Compare
geo2a
force-pushed
the
fraser-iohk/snapshot-config-sanity-checks
branch
3 times, most recently
from
0623697 to
1da8b89
Compare
jasagredo
approved these changes
Apr 23, 2026
geo2a
force-pushed
the
fraser-iohk/snapshot-config-sanity-checks
branch
from
1da8b89 to
b0281fb
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
As discussed in #1853, we want node operators to be warned if they configure their node in a way that is likely to be broken i.e. if they disable the snapshot rate limiting, or set the number of on-disk snapshots to 0, or set a snapshot interval that isn't a clean divisor of the epoch length (which is required for Mithril). This PR implements sanity checks similar to the existing ones for the
SnapshotPolicyArgsand traces warnings to the user if these guidelines are ignored.