Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Polkadot: Remove configuration's runtime migration and just set StorageVersion #4035

Merged
2 commits merged into from
Oct 7, 2021

Conversation

pepyakin
Copy link
Contributor

@pepyakin pepyakin commented Oct 7, 2021

This PR removes the runtime storage migration for parachain's configuration module. At the moment of writing, the migration was introduced by #3575 and is already in pipeline for Kusama.

For Polkadot we do not need to perform the migration. However, we do need to update the StorageVersion to sync it with the Kusama's one.

@pepyakin pepyakin added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Oct 7, 2021
Copy link
Contributor

@rphmeier rphmeier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@rphmeier
Copy link
Contributor

rphmeier commented Oct 7, 2021

bot merge

@ghost
Copy link

ghost commented Oct 7, 2021

Waiting for commit status.

@ghost
Copy link

ghost commented Oct 7, 2021

Merge aborted: Head SHA changed from b14f8cd to 1b1aed9

@s3krit
Copy link
Contributor

s3krit commented Oct 7, 2021

bot merge

@ghost
Copy link

ghost commented Oct 7, 2021

Trying merge.

@ghost ghost merged commit dd4b2e6 into master Oct 7, 2021
@ghost ghost deleted the pep-disable-host-config-migration branch October 7, 2021 16:02
s3krit pushed a commit that referenced this pull request Oct 7, 2021
@s3krit s3krit mentioned this pull request Oct 7, 2021
s3krit added a commit that referenced this pull request Oct 7, 2021
* v0.9.11 weights

* Allow an Offset to Lease Periods (#3980)

* add slot offset for slots

* trying things out

* fix test

* improve api to return the first block of a new lease period

* add an integration test with offset

* de-duplicate test

* hide lease period_period_length from public api

* fix benchmarks

* Update runtime/common/src/slots.rs

* support the exact same range of crowdloans

* fix docs

* fix docs again

* introduce offset to runtimes

* fix and check edge case w/ offset and lease period first block

* remove newline

* turn into an option

* fix benchmarks

Co-authored-by: Robert Habermeier <rphmeier@gmail.com>

* Polkadot: Remove configuration's runtime migration and just set StorageVersion  (#4035)

* wip

* Remove unused members

Co-authored-by: Shawn Tabrizi <shawntabrizi@gmail.com>
Co-authored-by: Robert Habermeier <rphmeier@gmail.com>
Co-authored-by: Sergei Shulepov <sergei@parity.io>
emostov pushed a commit that referenced this pull request Nov 1, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants