-
Notifications
You must be signed in to change notification settings - Fork 342
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
feat(testing): Task Sender / Stress Tester #1189
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.
It works. But it doesn't support many open connections. I think the idea is to open millions of concurrent requests. For that, we just need to open the socket connections, it is not necessary to send proofs which might be the bottleneck.
…a-or-createnewtaskerror-in-validityresponsemessage
…taskerror-in-validityresponsemessage' into 1015-task-sender
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.
Code looks good to me and it worked on devnet, I'll give you the approve after testing on staging.
Left you some nit comments, you don't have to address them
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.
Tested on staging, work as expected!
Note: I noticed that while sending proofs, after each iteration, the ws connections between the stress tester and the batcher are shown as broken. I think that this is a minor detail since it works fine.
@MauroToscano is the needed work in the SDK done? Maybe we can remove the change request? |
Note
This PR is pointed to 1314 branch because it needs the fix included in said branch.
PR of that branch is here
Changes
Adds a cli program to:
n
infinitely from the generated wallets.n
connection and hang infinitely.Testing
devnet
Note:
This will fund from every private key stored in the file
batcher/aligned-task-sender/wallets/devnet
You can take any amount of wallets you want from the file
batcher/aligned-task-sender/wallets/devnet-backup
This will create a
proofs
folder inscripts/test_files/task_sender/proofs
by default.Before running this command, you need to make sure you have first generated wallets and proofs. This command reads the default paths mentioned before.
BURST_SIZE
is how many proofs will each sender send.BURST_TIME_SECS
is how much time will each sender wait between each send_burst.This will start
NUM_SENDERS
connections and hang them infinitely without sending any proofTestnet
This will generate wallets on holesky, send them
AMOUNT_TO_DEPOSIT
funds fromFUNDING_WALLET_PRIVATE_KEY
, then sendAMOUNT_TO_DEPOSIT_TO_ALIGNED
funds from the newly created wallet to the BatcherPaymentService, to allow submission of proofs from this wallet.This will create a
proofs
folder inscripts/test_files/task_sender/proofs
by default.Before running this command, you need to make sure you have first generated wallets and proofs. This command reads the default paths mentioned before.
BURST_SIZE
is how many proofs will each sender send.BURST_TIME_SECS
is how much time will each sender wait between each send_burst.This will start
NUM_SENDERS
connections and hang them infinitely without sending any proofCloses #1113 and Closes #1015. Reopens #1162 due to merge conflicts.