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

Remove Kusama Nomination Pools Init #5605

Merged
merged 4 commits into from
Jun 16, 2022

Conversation

joepetrowski
Copy link
Contributor

Included in the runtime v9220 release, config setup is no longer needed.

I don't think pallet_nomination_pools::migration::v1::MigrateToV1<Runtime> is needed either but am not sure based on this thread https://github.com/paritytech/substrate/pull/11411/files#r872629403

@joepetrowski joepetrowski 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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels May 29, 2022
@paritytech-ci paritytech-ci requested review from a team May 29, 2022 19:05
@paritytech-ci paritytech-ci requested a review from a team May 30, 2022 11:24
Copy link
Member

@shawntabrizi shawntabrizi left a comment

Choose a reason for hiding this comment

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

lets also wait for kian

@kianenigma
Copy link
Contributor

Migrating pools to V1 is already in the 9.23 release, "making pool roles optional".

https://github.com/paritytech/polkadot/releases/tag/v0.9.23

@kianenigma
Copy link
Contributor

would be nice if you clean the stale migrations from westend too.

@shawntabrizi
Copy link
Member

bot rebase

@paritytech-processbot
Copy link

Rebasing

@shawntabrizi
Copy link
Member

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 98eb968 into master Jun 16, 2022
@paritytech-processbot paritytech-processbot bot deleted the joe-remove-nom-pools-init-kusama branch June 16, 2022 17:53
ordian added a commit that referenced this pull request Jun 17, 2022
* master:
  Get rid of unnecessary clone. (#5685)
  Remove Kusama Nomination Pools Init (#5605)
  Companion for paritytech/substrate#11631 (#5671)
  Allow relay chain council to send XCMs (#5597)
  Companion for substrate#11618 (#5677)
  Companion to combine iteratons and tolerance in sp-npos-elections API #11498 (#5581)
  Companion for #11661 (#5676)
  Bump lru from 0.7.5 to 0.7.7 (#5675)
  Companion PR for Substrate 11537 (#5619)
al3mart pushed a commit that referenced this pull request Jul 14, 2022
* remove kusama nomination pools init

* remove OnRuntimeUpgrade import

* remove stale migrations from westend

Co-authored-by: Shawn Tabrizi <shawntabrizi@gmail.com>
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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants