search

filecoin-project / oni

👹 (DEPRECATED; see README) Project Oni | Network Validation
https://docs.google.com/document/d/16jYL--EWYpJhxT9bakYq7ZBGLQ9SB940Wd1lTDOAbNE
7 stars 5 forks source link

fix deals stress tests #265

Closed nonsense closed 3 years ago

nonsense commented 3 years ago

Problem:

│ tg-lotus-soup-btuf5q1961nvkvepbg80-clients-1 {"ts":1602024283015012616,"msg":"","group_id":"clients","run_id":"btuf5q1961nvkvepbg80","event":{"message_event":{"message":"deal state: StorageDealCheckForAcceptance"}}}                    │
│ tg-lotus-soup-btuf5q1961nvkvepbg80-clients-1 2020-10-06T22:44:43.144Z    ERROR    storagemarket_impl    clientstates/client_states.go:289    deal bafyreiaaighyiszkeovze4dayvsjxqk7ecjpz6jfaaryxj7njxd4ltwt54 failed: deal failed: (State= │
│ 26) error calling node: publishing deal: GasEstimateMessageGas error: estimating gas used: message execution failed: exit 16, reason: Provider collateral out of bounds. (RetCode=16)                                                      │
│ tg-lotus-soup-btuf5q1961nvkvepbg80-clients-1 2020-10-06T22:44:43.166Z    ERROR    storagemarket_impl    clientstates/client_states.go:289    deal bafyreigdjj72upepll76z7j23qb4lhtpps4dvx3vktwcamtaomgtw47ykq failed: deal failed: (State= │
│ 26) error calling node: publishing deal: GasEstimateMessageGas error: estimating gas used: message execution failed: exit 16, reason: Provider collateral out of bounds. (RetCode=16)

This PR is amending the parameters used in the deals-e2e stress test. It looks like we are not setting up initial balances for clients and miners to support the number of concurrent deals we want to do, most probably due to the StartDeal parameters.

I am reducing a bit the concurrency factor and the number of clients creating deals with miners, so that the test passes, and then we can scale it up gradually, while keeping it passing.