IntersectMBO / cardano-node

The core component that is used to participate in a Cardano decentralised blockchain.
https://cardano.org
Apache License 2.0
3.07k stars 720 forks source link

Add more context to invalid snapshot events #5933

Closed amesgen closed 2 months ago

amesgen commented 3 months ago

Description

This PR adds a bit of context for when deserializing the ledger snapshot fails. Almost aways (apart from bugs/file corruption), this is due to an expected change in the serialization format, which currently requires replaying from Genesis. In the future, the Ledger team might implement migration/upgrade logic for this case, see https://github.com/IntersectMBO/cardano-ledger/issues/3220.

See #5908 for a recent example for users being puzzled by this message.

I opted to only modify the "human" representation, but I don't have a strong opinion here.

Checklist

Note on CI

If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons. You will need to get someone with write privileges. Please contact IOG node developers to do this for you.