Problem: query legacy block failed after upgrade

[yihuang]

Solution:

• fix in ethermint and update the dependency

👮🏻👮🏻👮🏻 !!!! REFERENCE THE PROBLEM YOUR ARE SOLVING IN THE PR TITLE AND DESCRIBE YOUR SOLUTION HERE !!!! DO NOT FORGET !!!! 👮🏻👮🏻👮🏻

PR Checklist:

• Have you read the CONTRIBUTING.md?
• Does your PR follow the C4 patch requirements?
• Have you rebased your work on top of the latest master?
• Have you checked your code compiles? (make)
• Have you included tests for any non-trivial functionality?
• Have you checked your code passes the unit tests? (make test)
• Have you checked your code formatting is correct? (go fmt)
• Have you checked your basic code style is fine? (golangci-lint run)
• If you added any dependencies, have you checked they do not contain any known vulnerabilities? (go list -json -m all | nancy sleuth)
• If your changes affect the client infrastructure, have you run the integration test?
• If your changes affect public APIs, does your PR follow the C4 evolution of public contracts?
• If your code changes public APIs, have you incremented the crate version numbers and documented your changes in the CHANGELOG.md?
• If you are contributing for the first time, please read the agreement in CONTRIBUTING.md now and add a comment to this pull request stating that your PR is in accordance with the Developer's Certificate of Origin.

Thank you for your code, it's appreciated! :)

[codecov]

Codecov Report

Merging #446 (9319573) into main (3ea70c5) will increase coverage by 17.88%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #446       +/-   ##
===========================================
+ Coverage   21.51%   39.40%   +17.88%     
===========================================
  Files          27       31        +4     
  Lines        1729     1571      -158     
===========================================
+ Hits          372      619      +247     
+ Misses       1324      906      -418     
- Partials       33       46       +13     

Impacted Files	Coverage Δ
x/cronos/genesis.go	61.53% <ø> (-38.47%)	⬇️
x/cronos/handler.go	100.00% <ø> (ø)
x/cronos/keeper/evm.go	58.00% <ø> (+6.27%)	⬆️
x/cronos/keeper/evm_hooks.go	80.00% <ø> (ø)
x/cronos/keeper/evm_log_handlers.go	83.50% <ø> (ø)
x/cronos/keeper/gravity_hooks.go	0.00% <ø> (ø)
x/cronos/keeper/grpc_query.go	0.00% <ø> (ø)
x/cronos/keeper/ibc.go	83.20% <ø> (+5.01%)	⬆️
x/cronos/keeper/ibc_hooks.go	50.00% <ø> (-8.83%)	⬇️
x/cronos/keeper/keeper.go	65.26% <ø> (-29.08%)	⬇️
... and 29 more

[yihuang]

https://github.com/crypto-org-chain/ethermint/compare/a178ce6dba6e...36b9c095b351?body=&expand=1

[tomtau]

why is there a need for the ethermint fork? can the upstream be used if the PRs are merged?

[yihuang]

why is there a need for the ethermint fork? can the upstream be used if the PRs are merged?

there's still breaking changes are not included for 0.7.0 upgrade, including the ibc-go 0.3, so we'll need to maintain this fork for the 0.7.0 release.