chore: rebase to upstream #11
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* modify logging * more * structure logs
* Add gas limit and DA transaction results for tracing * use result
* Add another builder tx after the first flashblock * Fix fb test
* Implement bundle reverting hashes * Merge main * Fix * Add min_block_number * fix litn * Fix * Fix comment * Mark invalid * Rename reverting_hashes * Remove Option * Revert "Remove Option" This reverts commit 4087d35.
…GE_BLOCKS (flashbots#128) If a user only specifies the min block number, the max block number is set to the current block number + MAX_BLOCK_RANGE_BLOCKS; however, there is no check that the min block is less than the newly set max block number.
* Fix bundle type reverting hashes optional param * Fix
Rebase Remove custom logic Custom Engine API support + engine call propagation
* Move bundle validation to primitives folder * Add another check * Fix
* Make op-rbuilder a lib * Fix lint * Two more fixes * Move alloc --------- Co-authored-by: Solar Mithril <[email protected]>
* Validation node works and tests are passing using the new test infra * revert_protection_monitor_transaction_gc passes * revert_protection_disabled passes * ordering tests are passing * all tests are passing * ready * lint * Github Action login to GHCR.io * testing Github Actions ghcr access * testing Github Actions ghcr access * lint * Closes issue 133 * review feedback * review feedback * Review feedback - ctrl-c * Review feedback * review feedback * Review feedback * review feedback * lint * macos compatibility * lint * lint
Logic Add extended flashblocks tracing
…bots#141) * Add minTimestamp and maxTimestamp as optional fields to bundle * Test that bundles with min timestamp wait aren't included until the block time exceeds it
* Remove extra generic * Fix fmt
* quick bugfix Account for dynamic lag in the beginning of building process * More improvements * More improvements * More improvements * More improvements * Wrap everything into the config * Review * Review
…lla builder (flashbots#145) * Applied 68 on top of latest main * Update crates/op-rbuilder/src/tests/smoke.rs Co-authored-by: Copilot <[email protected]> * lint * port new revert unit test * lint --------- Co-authored-by: Copilot <[email protected]>
…ts#163) * Add ci workflow to build independdent docker images + nightly * Update * More fixes
* record num txs built correctly with flashblocks enabled * Fix metrics --------- Co-authored-by: Solar Mithril <[email protected]>
* feat: add transaction gas limit * fix: update default * fix: update best txns * chore: add test for max txn gas * fix: apply suggestions * fix: tests * chore: update tests * fix: failing tests
* Add ExecutedBlock push on every built flashblock * Update crates/op-rbuilder/src/builders/flashblocks/payload.rs
* chore: namespace workflows per target * fix: strip `op-rbuilder` prefix from version tag
* chore: namespace workflows per target * fix: strip `op-rbuilder` prefix from version tag * fix: add quotes
* doc comments for bundle params * add check for flashblock number range * markdown doc
* Add doc regarding flow * Fix ordering * Update docs/flashblocks.md Co-authored-by: Julio <[email protected]> * Add comm --------- Co-authored-by: Julio <[email protected]>
* cli args * core impl * metrics * use the gas limiter * don't use optional gas limiter uses an "inner" struct to handle checking if it's enabled or not * cleanup stale addresses only every 100 blocks * unit tests * integration test
* refactor BundleOpts to use with_ methods to set * print logs during tests * fix connecting to flashblocks ws url * fix propagating bundle params for flashblocks check * refactor flashblock ws listener in tests * test setting min=max flashblock param
* bump reth * Add comm * Fix the problem
cody-wang-cb
force-pushed
the
cody/use_upstream
branch
2 times, most recently
from
f22509f to
d6cf5c7
Compare
cody-wang-cb
force-pushed
the
cody/use_upstream
branch
from
0fff379 to
1ec6310
Compare
* calculate state root * fix tests * add no state root test * check no tx pool instead * comments * optimize * fix lint * use no tx pool * add condition
cody-wang-cb
force-pushed
the
cody/use_upstream
branch
from
1ec6310 to
fed74d1
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Summary
flashbots#241
💡 Motivation and Context
✅ I have completed the following steps:
make lintmake test