Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

evm: in-process store migrations #601

Closed
fedekunze opened this issue Sep 28, 2021 · 2 comments · Fixed by #794
Closed

evm: in-process store migrations #601

fedekunze opened this issue Sep 28, 2021 · 2 comments · Fixed by #794
Labels
C:x/evm EVM module help wanted Extra attention is needed

Comments

@fedekunze
Copy link
Contributor

The EVM module is missing the in-process store migrations.

@fedekunze fedekunze added C:x/evm EVM module help wanted Extra attention is needed labels Sep 28, 2021
@yihuang
Copy link
Contributor

yihuang commented Nov 4, 2021

it seems EVM module's storage and parameters didn't change after 0.7.2 release, the only migration needed is adding the feemarket module, but that can be done in the app upgrade handler directly I think.
We should have it a test to confirm though.

@fedekunze
Copy link
Contributor Author

fedekunze commented Nov 8, 2021

@yihuang we should still add the command though. Otherwise, the genesis.json is going to be huge because of the contract code and storage. cc @tomtau @thomas-nguy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C:x/evm EVM module help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants