-
Notifications
You must be signed in to change notification settings - Fork 5
Stress test - too many bids - too many to finalize in one go #30
Conversation
Thanks, I merge. |
Did you have the patience to run it? |
I just ran it and it fails:
|
It may be because of truffle update which now count gas and make things slower. |
@stefek99 Can you look at this? |
I'm running it now, will finish in about 30 minutes... |
I'm with the new truffle version, that may be the difference. |
No way, I've updated recently!
|
Your screenshot still display ms. Mine now display the gas used. |
Are you at "truffle@4.1.11" ? |
That's |
Sh!t, it is failing now: |
Yeah, it's seems that the new truffle version is breaking stuff. |
See #33 Moved to a new folder so that Intermittent failure, really tricky to debug. |
It's not about increasing time. It's about running 5000 * 2 transactions so that finalize takes more than gasLimit of the block. |
@stefek99 |
Cannot set gasLimit on the mainnet. See #26 |
I know! Set lower gasLimit, start testrpc manually. Then we will hit it in 10 seconds, not half an hour... Thanks for suggestions! I won’t be investing more time on this issue unless explicitly instructed, I’m more interested in baking in affiliate / referral / commission to align incentives of participants as well as the core team when promoting the IICO. |
Some new lessons learned: 1. Cannot set block gas limit arbitrarily low - there will be not enough gas to deploy contract in the first place.Maybe there is a dynamic block gas limit somewhere, potentially? 2. Say I start it with
|
Output of the test: https://pastebin.com/XYQ51F0a
I left
console.log
statements on purpose so that it is possible to observe what's going on, feel free to remove as you see fit.It is send to a file, otherwise on error there were too many
Events
emitted and couldn't scroll the console.