Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enable Warp Sync #1585

Closed
wilwade opened this issue Jun 6, 2023 · 5 comments · Fixed by #1656
Closed

Enable Warp Sync #1585

wilwade opened this issue Jun 6, 2023 · 5 comments · Fixed by #1656
Assignees
@wilwade

Comments

@wilwade
Copy link
Collaborator

wilwade commented Jun 6, 2023 •
edited
Loading

Warp sync allows fast syncing and is currently not working. After #1270, we should be able to enable it.

Related:

Note that warp sync does not work on the Rococo Relay chain (although still works on the Frequency Testnet)
@rlaferla rlaferla self-assigned this Jul 31, 2023
@rlaferla
Copy link
Contributor

rlaferla commented Aug 1, 2023

https://spec.polkadot.network/chap-sync

@rlaferla
Copy link
Contributor

rlaferla commented Aug 1, 2023

How do we test warp sync to ensure integrity of data and consensus?

@wilwade
Copy link
Collaborator Author

wilwade commented Aug 1, 2023

The security is based on the relay chain peering. If we have a state with a root that matches one on the relay chain, then we know that state is good.

For warping the relay chain, it is almost the same, except it is that the additional blocks are built on the warp'd block.

While it is possible to chose a bad warp provider it is no different that choosing a bad chain spec or connecting to a bad bootnode.

@rlaferla
Copy link
Contributor

rlaferla commented Aug 2, 2023

It's not clear that Warp sync will work with Aura consensus but there are examples of it working with Grandpa. More research is needed.
https://substrate.stackexchange.com/questions/9484/how-to-enable-warp-sync-with-aura-consensus

@wilwade wilwade assigned wilwade and unassigned rlaferla Aug 10, 2023
@wilwade
Copy link
Collaborator Author

wilwade commented Aug 15, 2023 •
edited
Loading

Appears to be possible: paritytech/cumulus#1909

Yup added in v0.9.39: https://forum.polkadot.network/t/polkadot-release-analysis-v0-9-39/2277#add-warp-to-target-block-for-parachains-10

@wilwade wilwade mentioned this issue Aug 15, 2023
Merged
wilwade added a commit that referenced this issue Aug 17, 2023
@wilwade
Enable Warp Sync (#1656)
60b6436 
# Goal
The goal of this PR is to enable Warp sync for Frequency

Closes #1585 

# Discussion
- There are some additional updates possible to
`node/service/src/service.rs` that are possible once #1639 is complete
- Mostly this swaps out for the standard setup from cumulus

# Testing

1. `make build-mainnet-release`
2. `./target/release/frequency --tmp --sync warp -- --tmp --sync warp`
3. Look for `[Parachain] ⏩ Waiting for pending target block`
4. Wait until `[Parachain] ⏩ Block history`
5. Finally `[Parachain] ✨ Imported` (aka warp complete)
shannonwells pushed a commit that referenced this issue Apr 16, 2024
@wilwade @shannonwells
Enable Warp Sync (#1656)
2bb5ef5 
# Goal
The goal of this PR is to enable Warp sync for Frequency

Closes #1585 

# Discussion
- There are some additional updates possible to
`node/service/src/service.rs` that are possible once #1639 is complete
- Mostly this swaps out for the standard setup from cumulus

# Testing

1. `make build-mainnet-release`
2. `./target/release/frequency --tmp --sync warp -- --tmp --sync warp`
3. Look for `[Parachain] ⏩ Waiting for pending target block`
4. Wait until `[Parachain] ⏩ Block history`
5. Finally `[Parachain] ✨ Imported` (aka warp complete)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wilwade wilwade

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

1585-enable-warp-sync frequency-chain/frequency
Enable Warp Sync frequency-chain/frequency
2 participants
@rlaferla @wilwade