Skip to content

simulators/eels: fix execute-blobs simulator - #1365

Merged
spencer-tb merged 3 commits into
ethereum:masterfrom
spencer-tb:simulators/ethereum/eels/execute-blobs
Jul 30, 2026
Merged

spencer-tb merged 3 commits into
ethereum:masterfrom
spencer-tb:simulators/ethereum/eels/execute-blobs

Conversation

@spencer-tb

@spencer-tb spencer-tb commented Oct 28, 2025 •

Copy link
Copy Markdown
Collaborator

Description

The EEST execute command must be ran at the root of the EELS project following the Weld as that is where the ./tests are. This PR contains fixes for the latter.

Note cmake g++ libclang-dev are required to run uv sync --all-extras, note we are running this for the entire EELS repo now for execute and not just the EEST package.

Additionally the entrypoint now uses exec uv run --no-sync, matching #1577: the environment is synced once at image build, so container start skips the re-sync (a plain uv run would also re-sync without --all-extras, stripping packages installed at build).

This simulator runs the blob_transaction_test execute tests (engine_getBlobsV*), including the EIP-8070 getBlobsV4 tests from ethereum/execution-specs#2948 when built with fork=Amsterdam.

@lakshya-sky

Copy link
Copy Markdown

Hey @spencer-tb when do you think this will merged. I am having same problem running blob tests with hive.

@danceratopz danceratopz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have many issues running execute hive -m blob_transaction_test, but this isn't one of them ;) I don't think they're required any more.

Comment thread simulators/ethereum/eels/execute-blobs/Dockerfile Outdated
Comment thread simulators/ethereum/eels/execute-blobs/Dockerfile Outdated
Comment thread simulators/ethereum/eels/execute-blobs/Dockerfile
@danceratopz danceratopz self-assigned this Mar 11, 2026
@spencer-tb
spencer-tb merged commit dde4f59 into ethereum:master Jul 30, 2026
6 checks passed
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.

3 participants