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

Require MaxEncodedLen per default #10662

Merged
merged 8 commits into from
Jan 20, 2022

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Jan 14, 2022

  • Adds without_storage_info which is required on all Pallets that do not implement MaxEncodedLen
  • Removes generate_storage_info which is applied per default

Closes #10652
Related paritytech/polkadot-sdk#323

Polkadot companion: paritytech/polkadot#4746
Cumulus companion: paritytech/cumulus#917

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez added A3-in_progress Pull request is in progress. No review needed at this stage. B7-runtimenoteworthy C3-medium PR touches the given topic and has a medium impact on builders. labels Jan 14, 2022
@ggwpez ggwpez self-assigned this Jan 14, 2022
@github-actions github-actions bot added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels Jan 14, 2022
@ggwpez ggwpez added A3-in_progress Pull request is in progress. No review needed at this stage. and removed A0-please_review Pull request needs code review. labels Jan 14, 2022
@ggwpez
Copy link
Member Author

ggwpez commented Jan 14, 2022

Thanks @kianenigma for the very fast review!
I still want to extend the testing. Seems like the automation bot removed my inprogress label…

@ggwpez ggwpez marked this pull request as draft January 14, 2022 11:16
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez marked this pull request as ready for review January 14, 2022 13:25
@github-actions github-actions bot added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels Jan 14, 2022
@ggwpez ggwpez added A3-in_progress Pull request is in progress. No review needed at this stage. and removed A0-please_review Pull request needs code review. labels Jan 14, 2022
@shawntabrizi shawntabrizi added the D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit label Jan 14, 2022
@paritytech-processbot paritytech-processbot bot deleted the oty-without-storage-info branch January 20, 2022 11:16
wischli added a commit to KILTprotocol/kilt-node that referenced this pull request Jan 31, 2022
jasl added a commit to Phala-Network/khala-parachain that referenced this pull request Feb 6, 2022
jasl added a commit to Phala-Network/phala-blockchain that referenced this pull request Feb 6, 2022
jasl added a commit to Phala-Network/phala-blockchain that referenced this pull request Feb 6, 2022
jasl added a commit to Phala-Network/phala-blockchain that referenced this pull request Feb 7, 2022
jasl added a commit to Phala-Network/phala-blockchain that referenced this pull request Feb 7, 2022
jasl added a commit to Phala-Network/phala-blockchain that referenced this pull request Feb 7, 2022
jasl added a commit to Phala-Network/phala-blockchain that referenced this pull request Feb 7, 2022
jasl added a commit to Phala-Network/phala-blockchain that referenced this pull request Feb 7, 2022
wischli added a commit to KILTprotocol/kilt-node that referenced this pull request Feb 9, 2022
* chore: bump Polkadot dependencies to v0.9.16

* refactor: use AllPalletsWithSystem hook

* chore: remove crowdloan pallet

* fix: apply no DefaultAccount check

* fix: switch to default MaxEncodedLen

paritytech/substrate#10662

* fix: apply asset quota + runtime state_version

quota: paritytech/substrate#10382
state: paritytech/substrate#9732

* fix: Preimage registrar and Scheduler integration

Scheduler: paritytech/substrate#10356
OriginPriviligeCmp: paritytech/polkadot#4166

* fix: OrderedSet

* fix: apply new fork_id to chainspecs

paritytech/substrate#10463

* fix: apply no default account for SudoConfig

* fix: apply name changes for GrandPa

paritytech/substrate#10463

* fix: EnsureOneOf

paritytech/substrate#10379

* fix: preimage weights

* fix: parachain client

* fix: clippy copied weights

* fix: bump deps

* tests: attempt staking fix

* bench: attempt to fix default accountid for dids

* fix: staking unit test pallet order execution

* fix: did unit benchmarks

* fix: fmt

* fix: revert to old hook order execution

* bench: run manually for preimage + scheduler

* fix: only import TaskManager for try-runtime feature

* fix: use correct scheduler migration + add logs

* refactor: use explicit para runtime executors

* fix: apply code review by @Diiaablo95

* chore: apply suggestion from @weichweich review

* chore: bump deps

* fix: deps
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
* Remove generate_storage_info

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add without_storage_info where needed

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Update doc tests

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add more without_storage_info

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* fix TryBuild

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* fix TryBuild tests

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
AurevoirXavier added a commit to darwinia-network/darwinia-common that referenced this pull request Sep 14, 2022
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* Remove generate_storage_info

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add without_storage_info where needed

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Update doc tests

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add more without_storage_info

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* fix TryBuild

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* fix TryBuild tests

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C3-medium PR touches the given topic and has a medium 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.

Require MaxEncodedLen per default
4 participants