-
Notifications
You must be signed in to change notification settings - Fork 274
optimize contract test process #2284
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
base: devnet-ready
Are you sure you want to change the base?
Conversation
|
What was the reason for using a single node? Reason I ask is we were actually just thinking of increasing to 5 or so for e2e because mev shield has some edge cases that are only caught with an odd number of validators greater than 4 |
|
Just try to decrease the resource usage. Recently, I found out the process can't start or exit during e2e testing. If the e2e test can help with mev issue, then it makes sense to continue with two nodes testing. I will revert it to previous one, and optimize the process, make it easier to get what wrong during testing. |
Description
The changes in the PR include:
Related Issue(s)
Type of Change
Breaking Change
If this PR introduces a breaking change, please provide a detailed description of the impact and the migration path for existing applications.
Checklist
./scripts/fix_rust.shto ensure my code is formatted and linted correctlyScreenshots (if applicable)
Please include any relevant screenshots or GIFs that demonstrate the changes made.
Additional Notes
Please provide any additional information or context that may be helpful for reviewers.