-
Notifications
You must be signed in to change notification settings - Fork 16
Hstaykov/reorg detecting tasks per network #1750
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
Open
HristoStaykov
wants to merge
11
commits into
refactor_use_state_hash_for_sync_rebased
Choose a base branch
from
hstaykov/reorg_detecting_tasks_per_network
base: refactor_use_state_hash_for_sync_rebased
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Hstaykov/reorg detecting tasks per network #1750
HristoStaykov
wants to merge
11
commits into
refactor_use_state_hash_for_sync_rebased
from
hstaykov/reorg_detecting_tasks_per_network
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
db9f57d to
a30cd73
Compare
62da767 to
b314c2f
Compare
3548ead to
b7262b0
Compare
b314c2f to
2145c39
Compare
b7262b0 to
d1c5366
Compare
4a04194 to
617a135
Compare
00f9d71 to
e463124
Compare
e463124 to
ec4d8ae
Compare
617a135 to
fbc3456
Compare
ec4d8ae to
3615a5b
Compare
HristoStaykov
commented
Oct 28, 2025
| // It also verifies that a resync of indices is initiated when the on-chain root differs | ||
| // from the local calldata-merkle frontier (without deploying contracts). | ||
| #[tokio::test] | ||
| async fn test_loop_tracking_reorg_detect_and_resync_indices() { |
Contributor
Author
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Main reorg test
3615a5b to
a81ab7c
Compare
13bec09 to
a69c9ec
Compare
abe8d3f to
7566d8d
Compare
a69c9ec to
bb62a74
Compare
7566d8d to
35af4b0
Compare
Base automatically changed from
refactor_feed_id_stride
to
refactor_use_state_hash_for_sync_rebased
October 30, 2025 15:29
bb62a74 to
a03ddd3
Compare
5215d2f to
571691f
Compare
a03ddd3 to
3db5703
Compare
16eebd2 to
e906ed3
Compare
6a77f2d to
6f6555b
Compare
e906ed3 to
a7d3c68
Compare
6f6555b to
fc1cae8
Compare
a7d3c68 to
9809240
Compare
fc1cae8 to
8529704
Compare
d96ad89 to
93fa6c3
Compare
8529704 to
57ab0b4
Compare
93fa6c3 to
8f4ecd1
Compare
0551195 to
7c73bab
Compare
8f4ecd1 to
4fe7564
Compare
7c73bab to
90be292
Compare
140b9f8 to
eb670b4
Compare
90be292 to
0ceace1
Compare
…ized blocks - Wire per‑network reorg trackers: spawn loop_tracking_for_reorg_in_network via create_per_network_reorg_trackers - Refactor relayer path: eth_batch_send_to_contract now takes BatchOfUpdatesToProcess; update caller and use net_clone - Add InflightObservations to RpcProvider with helpers to insert_non_finalized_update, insert_observed_block_hash, prune_observed_up_to - Centralize await_time in blocksense_utils; remove duplicate in feed_registry and update tests/imports - Add imports for BlockNumberOrTag, B256, await_time, chrono::Local to support reorg tracking and timing - Improve task naming/log messages and fix minor comment typo
…k discovery, logging, and metrics; include a simulated‑reorg test that verifies resync
- feat(sequencer/providers): add reorg handling helper, counter, and test
- Introduce handle_reorg(...) in eth_send_utils to:
- Locate first common ancestor, log diverged blocks, and return fork height
- List non_finalized_updates at/above fork for visibility
- Add RpcProvider.reorgs_count with initialization to track detected reorgs
- Extend test flow to simulate a reorg by mining a new branch:
- Assert divergence at T1 and resync behavior (merkle_root_in_contract set, rb_indices refreshed)
- Verify reorgs_count increments and abort loop cleanly
- No changes to normal tx/receipt flow outside reorg handling paths
…cluded in blocks that have been reorg-ed
…ct reorg updates are processed
…new blocks are found
eb670b4 to
caee75b
Compare
0ceace1 to
dc50c60
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Testing