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

silkworm: tip execution with Erigon2 #12083

Open
wants to merge 18 commits into
base: release/2.60
Choose a base branch
from

Conversation

JacekGlen
Copy link
Member

@JacekGlen JacekGlen commented Sep 24, 2024

DO NOT MERGE YET - Waiting for clarification on Erigon2++ roadmap

This PR adds an option to run the tip execution with Erigon++. There are two entry points for silkworm:

  • verify chain
  • fork choice update

The changes to silkworm.go and backend.go follow the existing style and pattern.

Two extensive tests were added. silkworm_test.go focuses on the interface between Erigon and Silkworm. While ethereum_execution_test.go sets up the actual db and tests insert, validate chain and fcu functions. For both native Erigon and Erigon++.

@JacekGlen JacekGlen marked this pull request as ready for review September 24, 2024 14:35
@JacekGlen JacekGlen marked this pull request as draft October 8, 2024 17:44
@JacekGlen JacekGlen marked this pull request as ready for review October 8, 2024 18:05

func skipOnUnsupportedPlatform(t *testing.T) {
if !(runtime.GOOS == "linux" && runtime.GOARCH == "amd64") {
t.Skip("Silkworm is only supported on linux/amd64")
Copy link
Collaborator

@AskAlexSharov AskAlexSharov Oct 16, 2024

Choose a reason for hiding this comment

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

add check that silkworm is enabled

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.

2 participants