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

More flexible block fetching, and a few bug fixes. #1680

Merged
merged 30 commits into from
Oct 23, 2024

Conversation

rrw-zilliqa
Copy link
Contributor

@rrw-zilliqa rrw-zilliqa commented Oct 22, 2024

.. see the (rather long) commit list for a list of things this fixes.

We also add the ability for z2 to join nodes to a configured network, for debugging.

Closes #1364

@rrw-zilliqa rrw-zilliqa changed the title Fix a couple of bugs in block fetching More flexible block fetching, and a few bug fixes. Oct 22, 2024
(fix) Run the intershard comms test for 800 cycles - enough time for the node to sync
(fiX) Up the total ticks for joining shards; I've taken a look at a few scenarios and we only get out to 5 blocks with the new block sync code in 200 ticks. See previous comments re: optimising number of ticks.
(fix) Change name of constant to be more descriptive
…an easily change them, and bump the numbers to hopefully increase consistency.
(feat) Add prefixes to many trace calls
…r than panicking

(fix) When you have previously started with genesis, but not yet got a QC for it, start again - don't panic.
(fix) update commit-id for devnet deployment
(fix) zq2-devnet shouldn't overrun the total supply
@rrw-zilliqa rrw-zilliqa force-pushed the users/richard/1364-block-fetching-2 branch from 3a08f14 to f047ded Compare October 22, 2024 19:10
@rrw-zilliqa rrw-zilliqa enabled auto-merge October 23, 2024 08:34
@rrw-zilliqa rrw-zilliqa added this pull request to the merge queue Oct 23, 2024
Merged via the queue into main with commit e09283d Oct 23, 2024
5 checks passed
@rrw-zilliqa rrw-zilliqa deleted the users/richard/1364-block-fetching-2 branch October 23, 2024 09:48
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.

Checkpoints break the syncing algorithm in BlockStore
2 participants