sei-protocol / sei-chain

Apache License 2.0
2.7k stars 805 forks source link

Make occ caused evm panics less noisy #1728

Closed jewei1997 closed 3 months ago

jewei1997 commented 3 months ago

Describe your changes and provide context

If there is an evm panic due to occ conflict, don't log or increment metrics for this evm panic

Testing performed to validate your change

existing tests

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 60.65%. Comparing base (4f2c02d) to head (b8354b3). Report is 7 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/sei-protocol/sei-chain/pull/1728/graphs/tree.svg?width=650&height=150&src=pr&token=DG7ZZJ3PVV&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sei-protocol)](https://app.codecov.io/gh/sei-protocol/sei-chain/pull/1728?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sei-protocol) ```diff @@ Coverage Diff @@ ## main #1728 +/- ## ========================================== - Coverage 60.74% 60.65% -0.10% ========================================== Files 370 370 Lines 27433 27437 +4 ========================================== - Hits 16665 16641 -24 - Misses 9627 9654 +27 - Partials 1141 1142 +1 ``` | [Files](https://app.codecov.io/gh/sei-protocol/sei-chain/pull/1728?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sei-protocol) | Coverage Δ | | |---|---|---| | [x/evm/keeper/msg\_server.go](https://app.codecov.io/gh/sei-protocol/sei-chain/pull/1728?src=pr&el=tree&filepath=x%2Fevm%2Fkeeper%2Fmsg_server.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sei-protocol#diff-eC9ldm0va2VlcGVyL21zZ19zZXJ2ZXIuZ28=) | `76.17% <0.00%> (-0.28%)` | :arrow_down: | ... and [4 files with indirect coverage changes](https://app.codecov.io/gh/sei-protocol/sei-chain/pull/1728/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sei-protocol)