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

Add PoV-reclaim enablement guide to polkadot-sdk-docs #4244

Merged
merged 12 commits into from May 2, 2024

Conversation

skunert
Copy link
Contributor

@skunert skunert commented Apr 22, 2024

This adds a small guide on how to prepare your parachain for pov-reclaim usage.

@skunert skunert added R0-silent Changes should not be mentioned in any release notes T11-documentation This PR/Issue is related to documentation. labels Apr 22, 2024
@skunert skunert requested a review from kianenigma April 22, 2024 16:11
Copy link
Member

@ggwpez ggwpez left a comment

Choose a reason for hiding this comment

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

nice!

//! To reclaim excess storage weight, a parachain runtime needs the
//! ability to fetch the size of the storage proof from the node. The reclaim
//! mechanism uses the
//! [storage_proof_size](`cumulus_primitives_proof_size_hostfunction::storage_proof_size`)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm when I build locally, this link and other cumulus links below are broken, but I am unable to figure out why 🤔. Anyone got an idea? They look correct to me.

@skunert skunert requested a review from a team April 24, 2024 10:00
@dmitry-markin
Copy link
Contributor

CC @s0me0ne-unkn0wn

templates/parachain/node/Cargo.toml Show resolved Hide resolved
templates/parachain/runtime/Cargo.toml Show resolved Hide resolved
docs/sdk/src/guides/enable_pov_reclaim.rs Outdated Show resolved Hide resolved
docs/sdk/src/guides/enable_pov_reclaim.rs Outdated Show resolved Hide resolved
@kianenigma
Copy link
Contributor

kianenigma commented May 1, 2024

cc @SBalaguer this page can be useful for parachains to integrate this.

@skunert skunert added this pull request to the merge queue May 2, 2024
Merged via the queue into paritytech:master with commit b85c5a0 May 2, 2024
139 of 141 checks passed
@skunert skunert deleted the skunert/pov-reclaim-guide branch May 2, 2024 12:47
dcolley added a commit to metaspan/polkadot-sdk that referenced this pull request May 6, 2024
* 'master' of https://github.com/metaspan/polkadot-sdk: (65 commits)
  Introduces `TypeWithDefault<T, D: Get<T>>` (paritytech#4034)
  Publish `polkadot-sdk-frame`  crate (paritytech#4370)
  Add validate field to prdoc (paritytech#4368)
  State trie migration on asset-hub westend and collectives westend (paritytech#4185)
  Fix: dust unbonded for zero existential deposit (paritytech#4364)
  Bridge: added subcommand to relay single parachain header (paritytech#4365)
  Bridge: fix zombienet tests (paritytech#4367)
  [WIP][CI] Add more GHA jobs (paritytech#4270)
  Allow for 0 existential deposit in benchmarks for `pallet_staking`, `pallet_session`, and `pallet_balances` (paritytech#4346)
  Deprecate `NativeElseWasmExecutor` (paritytech#4329)
  More `xcm::v4` cleanup and `xcm_fee_payment_runtime_api::XcmPaymentApi` nits (paritytech#4355)
  sc-tracing: enable env-filter feature (paritytech#4357)
  deps: update jsonrpsee to v0.22.5 (paritytech#4330)
  Add PoV-reclaim enablement guide to polkadot-sdk-docs (paritytech#4244)
  cargo: Update experimental litep2p to latest version (paritytech#4344)
  Bridge: ignore client errors when calling recently added `*_free_headers_interval` methods (paritytech#4350)
  Make parachain template great again (and async backing ready) (paritytech#4295)
  [Backport] Version bumps and reorg prdocs from 1.11.0 (paritytech#4336)
  HRMP - set `DefaultChannelSizeAndCapacityWithSystem` with dynamic values according to the `ActiveConfig` (paritytech#4332)
  Statement Distribution Per Peer Rate Limit (paritytech#3444)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T11-documentation This PR/Issue is related to documentation.
Projects
Status: done
Development

Successfully merging this pull request may close these issues.

None yet

7 participants