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.

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

Enable Warp Sync #1656

Merged
merged 2 commits into from
Aug 17, 2023
Merged

Enable Warp Sync #1656

merged 2 commits into from
Aug 17, 2023

Conversation

wilwade
Copy link
Collaborator

@wilwade wilwade commented Aug 15, 2023

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 Upgrade to Polkadot Release v1.0.0 #1639 is complete
  • Mostly this swaps out for the standard setup from cumulus

Testing

  1. make build-mainnet-release
  2. ./target/release/frequency --base-path=/tmp/chain-warp --sync warp -- --sync warp
  3. Look for [Parachain] ⏩ Waiting for pending target block
  4. Wait until [Parachain] ⏩ Block history
  5. Finally [Parachain] ✨ Imported (aka warp complete)

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #1656 (d8d1219) into main (e35245e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1656   +/-   ##
=======================================
  Coverage   83.01%   83.01%           
=======================================
  Files          46       46           
  Lines        4332     4332           
=======================================
  Hits         3596     3596           
  Misses        736      736           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

log::warn!(
"⚠️ The hardware does not meet the minimal requirements for role 'Authority'."
);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is from the template. I liked it as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

I like it

Copy link
Collaborator

@saraswatpuneet saraswatpuneet left a comment

Choose a reason for hiding this comment

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

👍

@wilwade wilwade force-pushed the feature/warp-sync-1585 branch from bc5acf4 to dc9bea8 Compare August 15, 2023 17:57
Copy link
Collaborator

@shannonwells shannonwells left a comment

Choose a reason for hiding this comment

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

Works on my machine

Copy link
Contributor

@rlaferla rlaferla left a comment

Choose a reason for hiding this comment

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

Looks good!

@wilwade wilwade force-pushed the feature/warp-sync-1585 branch from dc9bea8 to d8d1219 Compare August 17, 2023 13:03
@wilwade wilwade merged commit 60b6436 into main Aug 17, 2023
@wilwade wilwade deleted the feature/warp-sync-1585 branch August 17, 2023 14:04
shannonwells pushed a commit that referenced this pull request Apr 16, 2024
# 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable Warp Sync
4 participants