Problem: estimate gas in evm tx is not accurate in relayer

CHANGELOG.md

@@ -13,6 +13,7 @@
 ### State Machine Breaking
 
 - [#1232](https://github.com/crypto-org-chain/cronos/pull/1232) Adjust require gas in relayer precompile to be closed with actual consumed.
+- [#1209](https://github.com/crypto-org-chain/cronos/pull/1209) Support accurate estimate gas in evm tx from relayer.
 - [#1247](https://github.com/crypto-org-chain/cronos/pull/1247) Update ethermint to develop, go-ethereum to `v1.11.2`.
 - [#1235](https://github.com/crypto-org-chain/cronos/pull/1235) Add channel detail in ica packet callback.
 
@@ -21,7 +22,6 @@
 - [#1239](https://github.com/crypto-org-chain/cronos/pull/1239) Refactor websocket/subscription system to improve performance and stability.
 - [#1241](https://github.com/crypto-org-chain/cronos/pull/1241) Improve parallelization of memiavl restoration.
 
-
 *October 17, 2023*
 
 ## v1.1.0-rc1

integration_tests/configs/ibc_rly.jsonnet

@@ -18,7 +18,7 @@ ibc {
   relayer+: {
     chains: [super.chains[0] {
       max_gas: 1000000,
-      gas_multiplier: 1.2,
+      gas_multiplier: 1.1,
     }] + super.chains[1:],
   },
 }

integration_tests/ibc_utils.py

@@ -267,7 +267,6 @@ def check_balance_change():
 
     wait_for_fn("balance change", check_balance_change)
     assert old_dst_balance + dst_amount == new_dst_balance
-
     # assert that the relayer transactions do enables the dynamic fee extension option.
     cli = ibc.cronos.cosmos_cli()
     criteria = "message.action=/ibc.core.channel.v1.MsgChannelOpenInit"
@@ -666,9 +665,6 @@ def log_gas_records(cli):
     records = []
     for tx in txs:
         res = tx["tx_result"]
-        actions = []
-        for event in res["events"]:
-            for attribute in event["attributes"]:
-                if attribute["key"] == "action":
-                    actions.append(attribute["value"])
-        records.append(res["gas_used"])
+        if res["gas_used"]:
+            records.append(res["gas_used"])
+    return records

integration_tests/test_ibc_rly_gas.py

@@ -21,11 +21,11 @@ def test_ibc(ibc):
     cli = ibc.cronos.cosmos_cli()
     wait_for_new_blocks(cli, 1)
     rly_transfer(ibc)
-    diff = 0.5
+    diff = 0.004
     record = log_gas_records(cli)
     if record:
         records.append(record)
     if len(records) == 2:
         for e1, e2 in zip(*records):
-            res = e2 / e1
+            res = float(e2) / float(e1)
             assert 1 - diff <= res <= 1 + diff, res

nix/default.nix

@@ -56,7 +56,7 @@ import sources.nixpkgs {
         name = "rly";
         src = sources.relayer;
         subPackages = [ "." ];
-        vendorSha256 = "sha256-IoaGLD3XKRmD61qcr/sPn3aWaa7zYAO9EbMiJFnF5BY=";
+        vendorSha256 = "sha256-5s5URiAZk59mOfZdUcqYAB/IvzFuBiQH5jDBuh6huTE=";
         doCheck = false;
         GOWORK = "off";
         postInstall = ''

nix/sources.json

@@ -126,10 +126,10 @@
         "homepage": "https://github.com/crypto-org-chain/relayer",
         "owner": "crypto-org-chain",
         "repo": "relayer",
-        "rev": "e097df316002173930a408cb5a7bce3040131c9c",
-        "sha256": "0xhgba9db9i56gglpgznpilkv0kb98m299yzgvpla60ldsd41imp",
+        "rev": "1432eb16fae943d7da2878893129cbf22406238d",
+        "sha256": "1i1270z9j68vpz0p4awl9v9br1cyp39qfwzsfy8vks1k33ibyzws",
         "type": "tarball",
-        "url": "https://github.com/crypto-org-chain/relayer/archive/e097df316002173930a408cb5a7bce3040131c9c.tar.gz",
+        "url": "https://github.com/crypto-org-chain/relayer/archive/1432eb16fae943d7da2878893129cbf22406238d.tar.gz",
         "url_template": "https://github.com/<owner>/<repo>/archive/<rev>.tar.gz"
     }
 }

scripts/gen-bindings-contracts

@@ -2,13 +2,15 @@
 
 solc08 --abi --bin x/cronos/events/bindings/src/CosmosTypes.sol -o build --overwrite
 solc08 --abi --bin x/cronos/events/bindings/src/Relayer.sol -o build --overwrite
+solc08 --abi --bin x/cronos/events/bindings/src/RelayerFunctions.sol -o build --overwrite
 solc08 --abi --bin x/cronos/events/bindings/src/Bank.sol -o build --overwrite
 solc08 --abi --bin x/cronos/events/bindings/src/ICA.sol -o build --overwrite
 solc08 --abi --bin x/cronos/events/bindings/src/ICACallback.sol -o build --overwrite
 
 
 abigen --pkg lib --abi build/CosmosTypes.abi --bin build/CosmosTypes.bin --out x/cronos/events/bindings/cosmos/lib/cosmos_types.abigen.go --type CosmosTypes
 abigen --pkg relayer --abi build/IRelayerModule.abi --bin build/IRelayerModule.bin --out x/cronos/events/bindings/cosmos/precompile/relayer/i_relayer_module.abigen.go --type RelayerModule
+abigen --pkg relayer --abi build/IRelayerFunctions.abi --bin build/IRelayerFunctions.bin --out x/cronos/events/bindings/cosmos/precompile/relayer/i_relayer_functions.abigen.go --type RelayerFunctions
 abigen --pkg bank --abi build/IBankModule.abi --bin build/IBankModule.bin --out x/cronos/events/bindings/cosmos/precompile/bank/i_bank_module.abigen.go --type BankModule
 abigen --pkg ica --abi build/IICAModule.abi --bin build/IICAModule.bin --out x/cronos/events/bindings/cosmos/precompile/ica/i_ica_module.abigen.go --type ICAModule
 abigen --pkg icacallback --abi build/IICACallback.abi --bin build/IICACallback.bin --out x/cronos/events/bindings/cosmos/precompile/icacallback/i_ica_callback.abigen.go --type ICACallback