Skip to content

docs: Fix state sync documentation#1363

Merged
martintomazic merged 3 commits intomainfrom
martin/fix/state-sync-documentation
Aug 25, 2025
Merged

docs: Fix state sync documentation#1363
martintomazic merged 3 commits intomainfrom
martin/fix/state-sync-documentation

Conversation

@martintomazic
Copy link
Copy Markdown
Contributor

@martintomazic martintomazic commented Jul 28, 2025

Document how to set correct trusted height and hash for the state sync so that they point to sufficiently old block header, to ensure there are available checkpoints.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 28, 2025

Deploy Preview for oasisprotocol-docs ready!

Name Link
🔨 Latest commit 9f411ae
🔍 Latest deploy log https://app.netlify.com/projects/oasisprotocol-docs/deploys/68ac5b1908faa800081a2653
😎 Deploy Preview https://deploy-preview-1363--oasisprotocol-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md
Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md Outdated
@martintomazic martintomazic force-pushed the martin/fix/state-sync-documentation branch from 2fb4ca0 to f77a601 Compare July 28, 2025 12:22
@martintomazic martintomazic marked this pull request as ready for review July 28, 2025 12:37
@martintomazic martintomazic requested review from kostko and matevz July 28, 2025 12:38
Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md Outdated
Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md Outdated
Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md Outdated
Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md Outdated
Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md Outdated
Copy link
Copy Markdown
Member

@matevz matevz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this can get merged and then we'll do followups. btw. Does ParaTime state sync work the same way?

@martintomazic martintomazic force-pushed the martin/fix/state-sync-documentation branch from f77a601 to a341e18 Compare July 31, 2025 15:19
@martintomazic
Copy link
Copy Markdown
Contributor Author

martintomazic commented Jul 31, 2025

I think this can get merged and then we'll do followups. btw. Does ParaTime state sync work the same way?

Sorry just saw this. I have added fixup commit explaining how to set correct trust headerperiod. Happy to remove also... :)

@martintomazic martintomazic requested a review from peternose July 31, 2025 15:23
@martintomazic
Copy link
Copy Markdown
Contributor Author

martintomazic commented Jul 31, 2025

Does ParaTime state sync work the same way?

It is simpler as you use reindexed history (roothash service). In a way consensus is light client for the runtimes, so you don't have this issues with trust period and sufficiently old checkpoints as for every consensus checkpoint checkpointer also creates runtime checkpoint.

The only think you can configure is disabling checkpoint sync and sync via diffs instead from the genesis I think.

Instead of genesis doc a trusted block header is used,
which implicitly defines trusted validator set used
by the light client protocol (I omitted this detail).

Finally, data is fetched from all nodes not just
full nodes.
@martintomazic martintomazic force-pushed the martin/fix/state-sync-documentation branch from a341e18 to e955ab6 Compare August 25, 2025 08:15
@martintomazic martintomazic requested a review from matevz August 25, 2025 08:20
Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md Outdated
Comment thread docs/node/run-your-node/advanced/sync-node-using-state-sync.md Outdated
@martintomazic martintomazic force-pushed the martin/fix/state-sync-documentation branch from e955ab6 to 2f48edd Compare August 25, 2025 11:49
@martintomazic martintomazic force-pushed the martin/fix/state-sync-documentation branch from 2f48edd to 9f411ae Compare August 25, 2025 12:46
@martintomazic martintomazic merged commit c7b1522 into main Aug 25, 2025
6 checks passed
@martintomazic martintomazic deleted the martin/fix/state-sync-documentation branch August 25, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants