EVM: Set FinalizedHash correctly in `forkChoiceUpdatedV3` call in ExecuteTxs
Maintainer antworten meist innerhalb von 2 Tagen
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- go
- Bereich
- blockchain
Rechercherichtung
Beginne in der EVM-Implementierung von ExecuteTxs beim ersten Aufruf von forkChoiceUpdatedV3 und lies anschließend PR 2394 sowie die im Issue erwähnten Startpfade von Rollkit Store und engineClient. Ermittle, ob der finalisierte Zustand aus DA Included Height oder aus persistierten Hashes stammen sollte, und definiere die Sanity Checks beim Start für die Hashes von head, safe und finalized. Als erledigt gilt die Aufgabe, wenn Neustarts und Indexer konsistente Block-Hashes erhalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
In https://github.com/rollkit/rollkit/pull/2394, in order to have sequencer restarts work well, we modified the first call to forkChoiceUpdatedV3 in the ExecuteTxs EVM implementation to assign the previous block hash to HeadBlockHash, SafeBlockHash, and FinalizedBlockHash.
We should revisit this and take one or more of the following approaches:
- Set the FinalizedBlockHash based on the DA Included Height from the Rollkit Store. For the Head and Safe block hashes, use the Rollkit height which should be the same as the reth block height but it might be good to actually sanity check that on startup and set them in engineClient correctly on start.
- Persist the current versions of head, safe, and finalizedBlockHashes somewhere so on restarts they can be loaded back easily.
The concrete consequences of setting finalizedHash incorrectly are unknown currently but it'll likely be at least an issue in indexers and block explorers showing inconsistent finalized block hashes for a small period of time.
- Vorherrschende Sprache
- Go
- Sterne
- 361
- Forks
- 283
- Ø Merge
- 3 T. 5 Std.
- Gemergte PRs (30 T.)
- 14
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus evstack/ev-node
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
Maintainer antworten meist innerhalb von 2 Tagen
-
C:p2p
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
Maintainer antworten meist innerhalb von 2 Tagen
-
good first issue T:enhancement
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 30/100
evstack/ev-node#1672 · 6 Kommentare ·
Maintainer antworten meist innerhalb von 2 Tagen
Alle Issues in evstack/ev-node
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
kind/engineering pulumi/pulumi-terraform Task Workflow Failure
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
pulumi/pulumi-terraform#1215 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug needs-acceptance wg/developer-experience-ecosystem
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
vllm-project/semantic-router#4480 ·
Maintainer antworten meist innerhalb von 1 Tag
-
area/docs kind/documentation priority/backlog triage/accepted
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
lexfrei/cloudflare-tunnel-gateway-controller#943 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
keyxmakerx/Chronicle#967 ·
Maintainer antworten meist innerhalb von 1 Tag