-
Notifications
You must be signed in to change notification settings - Fork 2.2k
feat(coprocessor): drift detection in gw-listener #2096
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
Merged
Merged
Changes from 42 commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
a36335f
feat(coprocessor): add opt-in drift detection to gw-listener
Eikix bfc8e11
fix(coprocessor): propagate consensus errors and defer unresolved dri…
Eikix 4ab48f2
refactor(coprocessor): trim drift detection to m1
Eikix afd309a
chore(coprocessor): drop m1 wording
Eikix c8b718f
feat(coprocessor): track drift variants and lagging coprocessors
Eikix 75b5598
refactor(coprocessor): fetch expected senders from gateway config
Eikix f1756f6
feat(coprocessor): rebuild drift tracking after restart
Eikix 6cd0a32
fix(coprocessor): silence drift replay on restart
Eikix af18faa
fix(coprocessor): keep consensus checks until local digests exist
Eikix 562d936
chore(coprocessor): apply rustfmt cleanup
Eikix 7ce03ed
Fix drift detector rebuild and sender refresh
Eikix ba11e62
Fix catch-up sender seeding and batch finalization
Eikix f7d804b
Wire drift detection into gw-listener chart
Eikix 56d267f
Bump coprocessor chart version
Eikix 3d776f7
Add ciphertext drift e2e workflow
Eikix c6f8795
Remove stray plan file
Eikix d114dac
Keep e2e drift test env intact
Eikix 7b05be2
Tighten drift listener and e2e injector
Eikix 84c4869
Use DB trigger for ciphertext drift e2e
Eikix 92f2b64
Fix drift handle completion and e2e exit path
Eikix 8308d66
test(e2e): assert ciphertext drift via listener logs
Eikix e1502dc
feat(gw-listener): add local attribution fields to drift logs
Eikix 18473d8
feat(gw-listener): expose drift tuning histograms
Eikix e8d529c
refactor(gw-listener): simplify drift detector and listener internals
Eikix 27237d6
test(drift-detector): add boundary and edge-case tests
Eikix aa6f456
refactor(tests): extract make_consensus_state helper, remove dead code
Eikix 962ec57
Fix review findings: metric pollution, DB waste, completion bug, e2e …
Eikix abeb07a
docs(gw-listener): clarify consensus invariant
Eikix ffc10a8
refactor(gw-listener): simplify drift detector helpers and listener d…
Eikix eb328f9
rename submit() to submit_digest_event_and_drift_check() in tests
Eikix 3f8a49b
refactor(gw-listener): simplify DriftDetector internals
Eikix 62a9be6
docs(gw-listener): document rebuild_drift_detector and facade methods
Eikix 898f0f6
fix(gw-listener): clarify drift counter description
Eikix 88449db
address review feedback: metrics docs, default timeouts, clap requires
Eikix b88641d
refactor(gw-listener): rename alerts_enabled to replaying (inverted)
Eikix dde3788
address review feedback: type alias, flush_metrics, tx-senders signature
Eikix 529fc88
refactor(gw-listener): rename HandleDisposition to HandleOutcome
Eikix d3c779a
address self-review: fix double-counted metrics, remove dead code, im…
Eikix eff20e0
fix(gw-listener): prevent drift detection errors from killing the lis…
Eikix 704897c
refactor(gw-listener): replace block-based drift timeouts with wall-c…
Eikix 5962e82
merge main into coproc-drift-heal
Eikix 996dd28
docs(gw-listener): update metrics docs for wall-clock timeouts, clari…
Eikix 26ddfd4
refactor(gw-listener): address review nits on drift detector
Eikix File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
3 changes: 3 additions & 0 deletions
3
...hevm-engine/db-migration/migrations/20260311154000_gw_listener_earliest_open_ct_block.sql
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| ALTER TABLE gw_listener_last_block | ||
| ADD COLUMN IF NOT EXISTS earliest_open_ct_commits_block BIGINT | ||
| CHECK (earliest_open_ct_commits_block >= 0); |
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.