Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lower Evm Max Priority #1463

Closed
wants to merge 199 commits into from
Closed

Lower Evm Max Priority #1463

wants to merge 199 commits into from

Conversation

Kbhat1
Copy link
Contributor

@Kbhat1 Kbhat1 commented Mar 22, 2024

Describe your changes and provide context

  • Lowers Max Priority for Evm Transactions to less than oracle and associate tx

Testing performed to validate your change

  • Updated unit test
  • Verifying on chain

codchen and others added 30 commits March 11, 2024 16:12
* evm wip

* Create EVM module and add types/keeper

* lint

* rebase

* tests

* add tests

* address comments
* evm wip

* Create EVM module and add types/keeper

* add tests

* EVM signature verification

* refactor
* [EVM] Add gas check ante handler

* add tests and remove unnecessary configs/versions
* [EVM] implement balance related methods for state interface

* add tests
)

* Add account state to EVM state interface implementation

* add Snapshot and rollback
* [EVM] Add gas check ante handler

* [EVM] implement balance related methods for state interface

* Add account state to EVM state interface implementation

* add Snapshot and rollback

* wip

* [EVM] add remaining state interface function

* rebase
* Add account state to EVM state interface implementation

* add Snapshot and rollback

* wip

* [EVM] add remaining state interface function

* [EVM] access list

* add unit tests

* improve code coverage

---------

Co-authored-by: Jeremy Wei <[email protected]>
* [EVM] implement message server that processes EVM transactions

* add tests
* [EVM] implement message server that processes EVM transactions

* [EVM] add bank precompile

* upgrade to Go 1.20

* rename coinbase

* tests & linter
* [EVM] implement message server that processes EVM transactions

* rename coinbase

* [EVM] add RPC/WS server helpers

* tests

* fix
* [EVM] implement message server that processes EVM transactions

* rename coinbase

* [EVM] add RPC/WS server helpers

* tests

* wip

* [EVM] add getBlockByHash endpoint

* rebase
[EVM] add endpoint to get block transaction count
* [EVM] add get transaction receipt endpoint

* refactor tests
[EVM] add getTransaction endpoints
[EVM] refactor gas logic
* [EVM] add endpoint to return fee history

* bump to 1.21
* Implement get code, storage, balance endpoints

* revert logging change

* also using bank keeper to check for balances

* appease linter
* simulate api

* add tests

* linter
* [EVM] add subscription utility

* finished new filter

* finished tests for new filter

* finished uninstall filter

* query tendermint for events

* in progress

* got first couple of tests done

* finished tests for get logs

* get filter logs and get filter changes

* refactor

* get filter logs accepts multiple addresses

* use multiple addresses and use cursor map

* remove bad test

* remove prints and todos

* minor fix

* fix lint errors and failing test

* poc to pass filtercriteria into JSON RPC

* made topics 2d array and adjusted regex

* GetLogs tests work

* Finished other functions

* remove unncessary inputs from NewFilterAPI

* add expiration/timeouts for filters

* removed error from UninstallFilter

* minor fix

* appease linter

---------

Co-authored-by: codchen <[email protected]>
* [EVM] return gas info in message handler correctly

* add comments
* integration

* fix tests
* [EVM] add transaction type to associate account

* tests
* [EVM] add transaction type to associate account

* [EVM] convert balance decimal points

* [EVM] add ACL dependencies for EVM transactions

* Revert "[EVM] add ACL dependencies for EVM transactions"

This reverts commit 5376a87.

* address comments
* added new block filter

* wip

* finished tests

* minor fix

* fix lint

* minor fix

* fix test
codchen and others added 11 commits March 19, 2024 09:38
* Set association and account for EVM contracts

* fix test
* EVM Associate Prefix Fix

* Remove debug

* Add Custom message field

* Custom Message

* remove recoverPubKey changes

* Update unit tests

* Update test message

* (Minor) test update
* EVM Precompile Tests

* Remove voting period reduction

* Rename file

* Revert gov change

* Update amount check
tick cosmos version to v0.2.78-seiv2
* loadtest throughput metrics

* fixes

* fix

* using per msg type on all metrics

* fix

* fix metrics keys and labels

* fix

* initialize maps

* add some prints

* fix defer

* fix

* fix

* go back to using normal map

* fix complilation

* fix lint
* Account for withdrawals in replay

* cancun prompt
@codecov-commenter
Copy link

codecov-commenter commented Mar 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.22%. Comparing base (0f9c32b) to head (c5747ca).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##            seiv2    #1463      +/-   ##
==========================================
- Coverage   62.32%   62.22%   -0.10%     
==========================================
  Files         352      352              
  Lines       24389    24389              
==========================================
- Hits        15201    15177      -24     
- Misses       8318     8346      +28     
+ Partials      870      866       -4     
Files Coverage Δ
app/antedecorators/priority.go 100.00% <100.00%> (ø)
x/evm/ante/fee.go 82.00% <100.00%> (ø)

... and 7 files with indirect coverage changes

@Kbhat1 Kbhat1 enabled auto-merge (squash) March 25, 2024 18:40
@Kbhat1 Kbhat1 disabled auto-merge March 25, 2024 18:45
@Kbhat1 Kbhat1 changed the base branch from seiv2 to evm April 4, 2024 13:22
@Kbhat1 Kbhat1 changed the base branch from evm to seiv2 April 4, 2024 13:23
@Kbhat1 Kbhat1 mentioned this pull request Apr 4, 2024
@Kbhat1
Copy link
Contributor Author

Kbhat1 commented Apr 4, 2024

Closing because we created this PR #1511 to deal with all the seiv2 rebase issues

@Kbhat1 Kbhat1 closed this Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants