Skip to content

feature: EELS backend #326

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 22 commits into from
Apr 22, 2025
Merged

feature: EELS backend #326

merged 22 commits into from
Apr 22, 2025

Conversation

@fselmo
Copy link
Contributor

@fselmo fselmo commented Apr 22, 2025 •
edited
Loading

What was wrong?

  • It would be a welcomed feature to be able to communicate with EELS as the backend. In the future, if a thin server can be wrapped around eth-tester, making calls to EELS could be used to test EIPs or any call against the official execution layer specifications.

How was it fixed?

  • This PR scrapes together an implementation for talking to EELS. It takes into account all type conversions and is working against the entire web3.py test suite. What it provably isn't is an efficient implementation. There are likely performance improvements that can be made on top of this initial implementation, as is evident with the core tests in web3.py taking ~30min to complete with the EELS backend.
  • Seeing that it's passing web3.py tests, and this is going into a feature branch, I would advocate for adding tests in a separate PR but could be convinced otherwise.

fselmo added 15 commits February 13, 2025 15:08
@fselmo
Initial implementation for EELS backend:
7bf42b1 
- Interact with accounts: get balance, nonce, code, storage at.
- Get block by hash or number.
- Send transactions, not all are tested yet... likely blob txs are not working.
- Still a lot of cleaning up to do.
@fselmo
Add initial support for pending, latest, earliest
dca0cc2
@fselmo
Broken but 'working' implementation of eth_getTransactionReceipt
68380d5
@fselmo
Initial implementation of eth_call
357eb23
@fselmo
Serialize logs; fix minor issues.
10075df
@fselmo
Properly generate a synthetic state with storage
f44a4b9
@fselmo
Fix mining blocks logic based on ghost genesis parent
f6885a1
@fselmo
Make sure to use the synthetic state for eth_call
b8af0b4
@fselmo
Fix get_storage() and beacon roots contract logic
227bb0a
@fselmo
Generate random prev randao and, for now, beacon parent root
9a10be9
@fselmo
Fix up snapshots and state retention logic
e792d82
@fselmo
Reconcile refactored differences across backends
c23d7c6
@fselmo
Add support for blob transactions.
cfbb5c4
@fselmo
Replace EELS exceptions with eth-tester exceptions
b1785f8
@fselmo
Reconcile differences across backends; some refactoring
34dec46
@fselmo fselmo marked this pull request as ready for review April 22, 2025 15:54
pacrob
pacrob approved these changes Apr 22, 2025
View reviewed changes
Copy link
Contributor

@pacrob pacrob left a comment

Choose a reason for hiding this comment

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

merge it!

fselmo added 7 commits April 22, 2025 17:09
@fselmo
Some refactoring; Fix transaction env to use latest mined block as nu…
30541d6 
…mber.

- Fix time travel / snapshot logic; update access list logic
@fselmo
y_parity clashes with v for legacy transactions
3d84b8c
@fselmo
Only process logs when mining; no receipt available otherwise
cf10720
@fselmo
Clean up / organize the EELSBackend implementation a bit; more fixes.
25dfb24 
- Refactor the block_number logic:
  - Use "pending" (current state) as the default state.
  - Use a context manager to handle state changes for the EELS backend.
  - Refactor the snapshotting logic to use a new class that keeps track
    of not only the state but also the chain data.

- Resolve some remaining issues found with web3.py integration testing
- Refactor reserved word filter->filter_; add blocknum logic to state methods
@fselmo
Resolve import issues; use forked repos for now
d94bcf4
@fselmo
Keep exceptions internal to eth-tester.
208e01f
@fselmo
Update eels version; reconcile new paths for eels types
a813e1b 
- Use newly-released eels pypi release as dep
- Last changes to account for EELS types errors in web3.py tests
- Fix issues with EELS types by forcing int when snapshotting
- bugfix: fix wrong types for get_storage_at
- Reconcile types with EELS; serialize as ints post-validation
@fselmo fselmo merged commit f8a6387 into ethereum:next Apr 22, 2025
7 of 29 checks passed
@fselmo fselmo deleted the eels-backend branch April 22, 2025 23:10
@fselmo fselmo restored the eels-backend branch April 22, 2025 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@pacrob pacrob pacrob approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fselmo @pacrob