-
Notifications
You must be signed in to change notification settings - Fork 185
chore: add osaka engine updates #1510
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
Conversation
This could even replace most of hive simulator tests if possible. The main issue right now is that most of the time we don't have full blobs in our tests but we only put the versioned hashes in the transaction. But perhaps we could put full blobs in the test transactions if we are running in execute mode (we could create a "mode" fixture that is populated with "fill" or "execute" depending on the pytest plugin). |
marioevz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this! Just a couple of comments.
Co-authored-by: Mario Vega <[email protected]>
Co-authored-by: Mario Vega <[email protected]>
marioevz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
* chore: add osaka engine updates. * chore(docs): add changelog. * Update src/ethereum_test_rpc/types.py Co-authored-by: Mario Vega <[email protected]> * Update src/ethereum_test_rpc/types.py Co-authored-by: Mario Vega <[email protected]> * chore: tox fixes. --------- Co-authored-by: Mario Vega <[email protected]>
* chore: add osaka engine updates. * chore(docs): add changelog. * Update src/ethereum_test_rpc/types.py Co-authored-by: Mario Vega <[email protected]> * Update src/ethereum_test_rpc/types.py Co-authored-by: Mario Vega <[email protected]> * chore: tox fixes. --------- Co-authored-by: Mario Vega <[email protected]>
🗒️ Description
Following: ethereum/execution-apis#630
I don't think we needed to change much on our end but please double check. I added the
get_blobsengine rpc method here on the assumption that we can maybe create a test suite that sends blob txs to ELs, then we callget_blobsto validate the response. This feels suited towards an extension inexecute.🔗 Related Issues
N/A.
✅ Checklist