feat: add maestro port
#384
Merged
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
daniilrrr
requested review from
Br1ght0ne,
WillPapper,
ericvelazquez,
ibremseth,
jorgemmsilva and
tsite
as code owners
SEQ-446 [spike] Evaluate Redis streams' viability for v1
AC:
|
daniilrrr
force-pushed
the
daniil/SEQ-446-n02-port
branch
from
3447469 to
c93c7c5
Compare
gustavoguimaraes
pushed a commit
that referenced
this pull request
Apr 2, 2025
* feat: use `shared` lib across repo (#378) * feat: use `shared` lib across repo * feat: `tc-client` using shared `json_rpc` * feat: `tc-client` using shared `json_rpc` * feat: use `shared` txn validation in sequencers * feat: check txn size in `validation` - use `Byte-Unit` lib for displaying txn sizes * feat: check txn size in `validation` - use `Byte-Unit` lib for displaying txn sizes * feat: remove dep * feat: move `test_utils` and `test_tracing` to `/shared` * feat: move `test_utils` and `test_tracing` to `/shared` - removes `metabased-translator/crates/test-utils` crate * feat: maestro Docker image (#383) * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: add `maestro` port (#384) * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: add redis_address * feat: add basic redis connection * fix: regenerate contract bindings to include walletpool module Co-Authored-By: gus@syndicate.io <gus@syndicate.io> --------- Co-authored-by: Daniil R <daniil@syndicate.io> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: gus@syndicate.io <gus@syndicate.io>
gustavoguimaraes
added a commit
that referenced
this pull request
Apr 7, 2025
* add feature for calldata check modules * create module for dream chain calldata usecase * add test for CalldataAlwaysAllwedModule * rust bindings * Add test for DreamChainCheckCallerFromCalldataModule * fix ftm error * Fix merge conflicts in PR #346 (#351) * remove unused RESTORE_FROM_SAFE_STATE flag (#338) * remove _CHAIN_ from env vars (#340) * docs: Add documentation for forge doc command (#341) * Add documentation for forge doc command Co-Authored-By: will@syndicate.io <will@syndicate.io> * Update forge doc documentation with default behavior Co-Authored-By: will@syndicate.io <will@syndicate.io> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: will@syndicate.io <will@syndicate.io> * Remove BLOCK_BUILDER_ and SLOTTER_ prefixes from config values (#339) * Remove BLOCK_BUILDER_ and SLOTTER_ prefixes from config values Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update variable names to match environment variable names Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update references to renamed variables Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update test cases to use renamed variables Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Apply cargo fmt to fix formatting issues Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update variable references in arbitrum_adapter.rs Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update variable references in full_meta_node.rs test file Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update ignore_delayed_messages to arbitrum_ignore_delayed_messages in tests Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update owner_address to rollup_owner_address in integration_tests.rs Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Add empty rustfmt config to fix formatting Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update rustfmt config to specify edition Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update rustfmt config with max_width Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Update rustfmt config with additional settings Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Remove .rustfmt.toml as requested Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * Run cargo +nightly fmt directly as requested Co-Authored-By: jorge@syndicate.io <jorge@syndicate.io> * fix tests * ROLLUP_TYPE -> TARGET_ROLLUP_TYPE --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: jorge@syndicate.io <jorge@syndicate.io> * Update pr template to indicate metrics changes (#342) * feat: maestro skeleton (#344) * feat: maestro skeleton * feat: metabased-poster v1 (#337) * feat: metabased poster scaffolding * feat: metabased-poster poller * poller working * update contract * submitter working * update submitter * update config * update submitter * test * rm rusttool * update logger * simple poster * simple poster * simplify poster * simplify poster * SEQ-694: update test to use assertion poster contract (#343) Ticket Related Linear Ticket: SEQ-694 What does this PR do? Summary: add a contract for posting assertions that the assertion poster (feat: metabased-poster v1 #337) uses Key Changes: remove unused forge-std submodule reorganize contracts folder layout to separate the arbitrum interfaces from our contracts Breaking changes? Yes/No: No Metrics changes? Yes/No: No Anything interesting? All existing metabased rollups can be upgraded to support withdrawals by deploying & initializing the AssertionPoster contract! How can this PR be tested? ci tests * update test to check for config changes (#349) Ticket Related Linear Ticket: SEQ-723 What does this PR do? Summary: update withdrawal test to check that config changes are handled correctly Breaking changes? Yes/No: No How can this PR be tested? manually tested to make sure the test fails if the config values are not updated in the contract ci tests * feat: poster docker & deployment scripts (#350) * feat: poster doker & deployment scripts * typo * Fix remaining merge conflicts in arbitrum binding files Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Update Rust contract bindings from feature branch Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Fix missing iassertionposter module in arbitrum bindings Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Add missing module declarations to arbitrum/mod.rs Co-Authored-By: gus@syndicate.io <gus@syndicate.io> --------- Co-authored-by: Jorge Silva <jorge@syndicate.io> Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: will@syndicate.io <will@syndicate.io> Co-authored-by: Kristina Fischer <158780688+squibwarb@users.noreply.github.com> Co-authored-by: Daniil R <daniil@syndicate.io> Co-authored-by: Eric Velazquez <ericvelmar95@gmail.com> Co-authored-by: tsite <stubydent@gmail.com> Co-authored-by: gus@syndicate.io <gus@syndicate.io> * rust bindings * Fix missing forge-std submodule URL for metabased-translator/contracts (#353) Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: gus@syndicate.io <gus@syndicate.io> * update Certora * fix MetabasedSequencerChain certora spec * Certora fixez for RequireAllModule * add spec for RequireAnyModule * revert certora version to see if it works * downgrade python version * rust bindings * Fix CI pipeline: update Rust contract bindings and fix Certora installation (#354) Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: gus@syndicate.io <gus@syndicate.io> * install setuptools==75.8.2 before certora * fix spec * delete superfluous file * Add WalletPoolSequencingModule * rust bindings * add WalletPoolSequencingModule to certora CI job * Fix CI issues in WalletPoolSequencingModule (#358) * Fix CI by updating test environment to configure WalletPoolSequencingModule Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Fix formatting issues in full_meta_node.rs Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Fix Clippy warning by prefixing unused variable with underscore Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Add test addresses to wallet pool to fix integration tests Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Update contract bindings to fix formatting issues Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Fix variable name and formatting issues in full_meta_node.rs Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Fix settlement_provider reference by using self.settlement_provider Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Fix settlement provider reference in test addresses array Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Fix provider reference in test addresses array Co-Authored-By: gus@syndicate.io <gus@syndicate.io> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: gus@syndicate.io <gus@syndicate.io> * Fix WalletPool based on PR suggestions * fix certora CI job * rust bindings * fix: update walletpoolsequencingmodule references to use walletpool Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * fix: update addToWalletPool call with missing parameter Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * fix: apply rustfmt formatting Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Fix CI errors in PR #357 (#385) * feat: use `shared` lib across repo (#378) * feat: use `shared` lib across repo * feat: `tc-client` using shared `json_rpc` * feat: `tc-client` using shared `json_rpc` * feat: use `shared` txn validation in sequencers * feat: check txn size in `validation` - use `Byte-Unit` lib for displaying txn sizes * feat: check txn size in `validation` - use `Byte-Unit` lib for displaying txn sizes * feat: remove dep * feat: move `test_utils` and `test_tracing` to `/shared` * feat: move `test_utils` and `test_tracing` to `/shared` - removes `metabased-translator/crates/test-utils` crate * feat: maestro Docker image (#383) * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: add `maestro` port (#384) * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: maestro Docker image * feat: add redis_address * feat: add basic redis connection * fix: regenerate contract bindings to include walletpool module Co-Authored-By: gus@syndicate.io <gus@syndicate.io> --------- Co-authored-by: Daniil R <daniil@syndicate.io> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: gus@syndicate.io <gus@syndicate.io> * Undo changes in full_meta_node.rs as requested Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Align full_meta_node.rs with main branch Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * Revert full_meta_node.rs to match main branch exactly Co-Authored-By: gus@syndicate.io <gus@syndicate.io> * add WalletPoolWrapperModule * rb * bindings * update bindings * refactor WalletPoolWrapperModule * add WalletPoolWrapperModule spec * delete obsolete WalletPool.sol * rb * refactor WalletPoolWrapperModule * add to certora spec * rb --------- Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Jorge Silva <jorge@syndicate.io> Co-authored-by: will@syndicate.io <will@syndicate.io> Co-authored-by: Kristina Fischer <158780688+squibwarb@users.noreply.github.com> Co-authored-by: Daniil R <daniil@syndicate.io> Co-authored-by: Eric Velazquez <ericvelmar95@gmail.com> Co-authored-by: tsite <stubydent@gmail.com> Co-authored-by: gus@syndicate.io <gus@syndicate.io>
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.
Ticket
What does this PR do?
maestro