op-e2e: test op-geth pending gas limit configuration#4047
Conversation
|
|
Converted this into draft. It's ready, but not that urgent. Happy thanksgiving. |
|
Hey @protolambda! This PR has merge conflicts. Please fix them before continuing review. |
4577199 to
0195cf9
Compare
|
Ready for review, but we'll need to update the op-geth references in |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #4047 +/- ##
===========================================
- Coverage 41.11% 35.26% -5.85%
===========================================
Files 322 170 -152
Lines 17526 13357 -4169
Branches 784 0 -784
===========================================
- Hits 7205 4711 -2494
+ Misses 9808 8167 -1641
+ Partials 513 479 -34
Flags with carried forward coverage won't be shown. Click here to find out more. |
tynes
left a comment
There was a problem hiding this comment.
Looks good to me with a couple of questions
|
*need to merge op-geth PR + update mod references before merging this PR. |
|
merged op-geth PR, will update remaining geth references now for this PR to get merged with those |
|
updated op-geth with |
Description
Test the pending gas limit configurability in ethereum-optimism/op-geth#24
TODO: merge the op-geth PR first, and then update the geth references everywhere. For now I updated only the
go.workfor CI to run.Tests
Test that blocks can be produced, and the verifier can pick up blocks, with consistent block gas limit that does not conflict with the local sequencer or verifier pending gas limit configurations.
Additional context
Reducing the local pending gas limit helps prevent self-DoS when the tx pool fills up and the gas limit is extremely high (e.g. the opcraft chain)
Metadata
Fix ENG-2978