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

Taurus storage migration for "Private EVM" #3360

Draft
wants to merge 6 commits into
base: private-evm
Choose a base branch
from

Conversation

teor2345
Copy link
Member

@teor2345 teor2345 commented Jan 28, 2025

TODO

  • Change the merge base to taurus-runtime-upgrade

How to review this PR

This PR is the third in a series of PRs which add "private EVM" functionality to subspace. It upgrades the existing Taurus storage format for this feature. (Mainnet doesn't need an upgrade, because it doesn't have any EVMs deployed.)

The PR diff includes the migration from #3342, please ignore the migration module and VersionCheckedMigrateDomainsV1ToV2 type.

The first PR in the series is #3359, it moves some test code around, and makes minor fixes.

The second PR in this series is #3350, which adds the feature to Mainnet.

What it does

This PR upgrades the domain registry storage into a new format which stores the initial contract creation allow list.

It also merges all the changes in #3350 into the taurus-runtime-upgrades branch.

Code contributor checklist:

@teor2345 teor2345 added execution Subspace execution breaking-runtime This PR introduces breaking changes to the runtime labels Jan 28, 2025
@teor2345 teor2345 self-assigned this Jan 28, 2025
@teor2345 teor2345 changed the title Taurus migration for "Private EVM" Taurus storage migration for "Private EVM" Jan 28, 2025
@teor2345 teor2345 force-pushed the private-evm-taurus-migration branch from 29655a9 to a425f09 Compare January 28, 2025 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-runtime This PR introduces breaking changes to the runtime execution Subspace execution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants