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

feat(runtimes): update devnet and testnet runtimes to v1.14.0 #171

Closed
wants to merge 1 commit into from

Conversation

peterwht
Copy link
Collaborator

@peterwht peterwht commented Aug 9, 2024

Here is a demonstrative PR showing how uplifting runtimes is relatively easy. This is not a comprehensive PR and should not be treated as 100% accurate.

This does a jump of 7-versions. The changes in the runtime files are very minimal, mostly configuration changes, small changes to the runtime-apis.

The most difficult change (which needs to be analyzed more closely) was with how pallet-contracts exposes host_fn_weights.

Only the runtimes build:

cargo build -p pop-runtime-devnet
cargo build -p pop-runtime-testnet

Updating the runtime took only 30 minutes excluding the chain extension change. This is because rust outputs very useful compiler errors for the runtime, which makes it super easy to spot what needs fixing.

The node however spits out a large wall of errors that take much more analysis. If this was an OmniNode, we would have to waste very little brain energy.
Screenshot 2024-08-09 at 2 54 32 PM

@peterwht peterwht closed this Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant