Results 84 comments of Tanmay

v21 modifies the `UpdateChainInfo` message , and is a breaking change We should verify on athens 3 , that we do not run into a similar issues as this https://github.com/zeta-chain/node/pull/2890

Tests for athens v34 # Cosmos SDK Modules and Messages ## Authz - [x] MsgExec `zetaclient trasnactions are all executed through MsgExec` - [x] MsgGrant `zqt tx 44789A665B3461D7B9050938D3D892BCC9DB975C8EB08E590FE8F5F3A365673E` ## Bank...

Updated the title Based on some investigation, I feel like the main cause of things breaking post upgrade seems to be the divergence in types. These tests should be separate...

This can be used to run tests on a node that has all the testnet/mainnet state. It would enable us to replace the state export tests. (Which are currently blocked...

Related https://github.com/zeta-chain/node/issues/3803

All errors are eventually logged The logging happens either at the function itself, like the SUI or TON examples, or they get logged at Task level Block ticker ``` if...

Update based on discusion with @renan061 The core idea is to log/store important debug data in a storage, (most likely a key-value pair ) so that its easier to find...

The naming used for the current fix might be a bit misleading. ``` isWithdrawWithNoCall := !cctx.IsCurrentOutboundRevert() && cctx.InboundParams.IsCrossChainCall isRevertWithNoCall := cctx.IsCurrentOutboundRevert() && cctx.RevertOptions.CallOnRevert ``` Since we are checking the limit...

The low gas limit check on the client is not needed anymore, we already check the gas limit before creating the outbound https://github.com/zeta-chain/protocol-contracts/blob/48ecdbcc4c30a8e8a7f2f5756b6a544b316670a0/contracts/zevm/libraries/GatewayZEVMValidations.sol#L40-L42 Also In my opinion the interinsic gas...

We can rule out intrinsic gas too low as an issue , however the current scheduler by design would keep retrying if for some reason the broadcast or validation fails...