forest icon indicating copy to clipboard operation
forest copied to clipboard

fix `Filecoin.GasEstimateMessageGas`

Open ansermino opened this issue 1 year ago • 3 comments

As reported in the curio-dev channel here:

Seems like forest gas estimation is acting up - My Calibnet WindowPoSts failed to send with

error: sending proof message: send error: RPC error (-32603): given message has too high of a gas limit

Snap later failed with

error: pushing message to mpool: send error: RPC error (-32603): gas_limit 0 cannot be less than cost 4958563.000 of storing a message on chain

(lotus-backed sends failed with error: pushing message to mpool: GasEstimateMessageGas error: message execution failed: exit ErrIllegalArgument(16), reason: message failed with backtrace: 00: f0116147 (method 35) -- invalid update 0 while requiring activation success: sector isn't active, skipping sector 29 (16) (RetCode=16) so this one is probably not returning the correct error in Gas Estimation)

Sadly, we have no snapshots of the invalid behaviour.

ansermino avatar Aug 21 '24 14:08 ansermino

Updated the issue - the method is implemented, but something is not working right.

LesnyRumcajs avatar Aug 22 '24 09:08 LesnyRumcajs

Blocked by https://github.com/ChainSafe/forest/issues/4716

LesnyRumcajs avatar Sep 02 '24 09:09 LesnyRumcajs

Can't replicate right now.

LesnyRumcajs avatar Oct 08 '24 09:10 LesnyRumcajs

@LesnyRumcajs Please confirm this is unblocked and remove the label if so

ansermino avatar Nov 08 '24 05:11 ansermino

@ansermino It is not unblocked because it is not actionable given we don't have means of reproducing this. We are waiting on Curio devs on this, given it happen only on their side.

LesnyRumcajs avatar Nov 08 '24 06:11 LesnyRumcajs

Closing. We can re-open when (if) the issue is confirmed.

LesnyRumcajs avatar Dec 05 '24 11:12 LesnyRumcajs