-
Notifications
You must be signed in to change notification settings - Fork 75
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
Test startup for more variations of command line args #78
Conversation
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.
Looks good, there are some more L3 related flags which are used in CIs across different repos and have caused previousy crashes/issues: --l3-fee-token --l3-token-bridge --l3-fee-token-decimals 6
. We don't need to test every possible combo with these, but I think at least one run with all of them would be useful have
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
* Update migration test for TEE integration This commit updates the migration-test.bash script to reflect the new migration flow for the TEE integration. It also adds some utility scripts and updates some exsisting example files to assist in the migration test. * Remove changes to docker-compose.yml * Respond to review feedback * Update migration test to be compatible with integration branch * Fix the batch poster In this commit, a dummy TEE Verifier address is being used, we will use a correct one later * Fix config.ts to allow sequencer node to start * Update orbit-actions sub-module to integration branch * update migration test to point at integration branch * Fix smoke test --------- Co-authored-by: ImJeremyHe <[email protected]>
No description provided.