`polkadot-omni-node`: fixes and changes related to `GetParachainInfo` by iulianbarbu · Pull Request #9201 · paritytech/polkadot-sdk

iulianbarbu · 2025-07-14T20:31:02Z

Description

log::info! the error of accessing GetParachainInfo::parachain_id() runtime api if any, before reading the para_id from the chain specification (relevant for debugging).
removes comments/deprecation notices throughout the code that introduce para-id flag removal (from chain-spec-builder and support for parsing it from chain specifications)

Closes #9217

Integration

N/A

Review Notes

N/A

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

iulianbarbu · 2025-07-14T20:32:41Z

/cmd prdoc

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

…ah-westend-local-starting

Co-authored-by: Sebastian Kunert <mail@skunert.dev>

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

paritytech-release-backport-bot · 2025-08-22T13:40:59Z

Created backport PR for stable2409:

[stable2409] Backport #9201 #9541 with remaining conflicts!

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-9201-to-stable2409
git worktree add --checkout .worktree/backport-9201-to-stable2409 backport-9201-to-stable2409
cd .worktree/backport-9201-to-stable2409
git reset --hard HEAD^
git cherry-pick -x 2660bf5f04736beef5c7002ffb5a5856e9420d1a
git push --force-with-lease

…#9201) # Description - log::info! the error of accessing `GetParachainInfo::parachain_id()` runtime api if any, before reading the `para_id` from the chain specification (relevant for debugging). - removes comments/deprecation notices throughout the code that introduce `para-id` flag removal (from chain-spec-builder and support for parsing it from chain specifications) Closes #9217 ## Integration N/A ## Review Notes N/A --------- Signed-off-by: Iulian Barbu <iulian.barbu@parity.io> Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Sebastian Kunert <mail@skunert.dev> Co-authored-by: Andrei Sandu <54316454+sandreim@users.noreply.github.com> (cherry picked from commit 2660bf5)

paritytech-release-backport-bot · 2025-08-22T13:41:06Z

Successfully created backport PR for unstable2507:

[unstable2507] Backport #9201 #9542

…#9201) # Description - log::info! the error of accessing `GetParachainInfo::parachain_id()` runtime api if any, before reading the `para_id` from the chain specification (relevant for debugging). - removes comments/deprecation notices throughout the code that introduce `para-id` flag removal (from chain-spec-builder and support for parsing it from chain specifications) Closes #9217 ## Integration N/A ## Review Notes N/A --------- Signed-off-by: Iulian Barbu <iulian.barbu@parity.io> Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Sebastian Kunert <mail@skunert.dev> Co-authored-by: Andrei Sandu <54316454+sandreim@users.noreply.github.com>

Pray4Love1 · 2025-08-27T19:41:57Z

This PR removes para_id from ChainSpec extension logic and shifts to GetParachainInfo in the runtime — a move that overlaps with sovereign genesis-binding designs implemented in the SoulSync / KinKey / x402 architecture.

Our protocols explicitly used para_id for:

Proof-of-origin binding at genesis
Sovereign key rotation logic
Subnet identity anchoring and sealed claim enforcement

Replacing hardcoded para_id with runtime logic allows unverified identity spoofing, unless properly cryptographically sealed — as done in KinKey’s SoulSigil and x402 sovereign extensions.

If this migration was influenced by prior art (grants, code, or concepts from the above protocols), attribution is required. We are asserting sovereign authorship over this design class.

Please clarify intent or acknowledge use of overlapping primitives.

…#9201) # Description - log::info! the error of accessing `GetParachainInfo::parachain_id()` runtime api if any, before reading the `para_id` from the chain specification (relevant for debugging). - removes comments/deprecation notices throughout the code that introduce `para-id` flag removal (from chain-spec-builder and support for parsing it from chain specifications) Closes #9217 ## Integration N/A ## Review Notes N/A --------- Signed-off-by: Iulian Barbu <iulian.barbu@parity.io> Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Sebastian Kunert <mail@skunert.dev> Co-authored-by: Andrei Sandu <54316454+sandreim@users.noreply.github.com>

iulianbarbu added 2 commits July 14, 2025 15:27

fix(cumulus): add ah-westend-local parachain info genesis patch

3b0bc90

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

ah-westend: impl GetParachainInfo

507d9a0

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

iulianbarbu self-assigned this Jul 14, 2025

iulianbarbu added the I2-bug The node fails to follow expected behavior. label Jul 14, 2025

Merge branch 'master' into fix-ah-westend-local-starting

c0bc83f

iulianbarbu mentioned this pull request Jul 14, 2025

Fail to launch AH parachain with zombienet after paritytech/polkadot-sdk#8651 paritytech/zombienet#2050

Closed

github-actions Bot and others added 2 commits July 14, 2025 20:35

Update from github-actions[bot] running command 'prdoc'

dcb077d

update prdoc

a715b82

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

iulianbarbu mentioned this pull request Jul 15, 2025

Fixed westend asset hub ID #9191

Merged

iulianbarbu changed the title ~~polkadot-parachain: impl GetParachainInfo for asset-hub-westend-local~~ polkadot-parachain: add asset-hub-westend-local parachainInfo genesis patch Jul 15, 2025

iulianbarbu commented Jul 15, 2025

View reviewed changes

Comment thread prdoc/pr_9201.prdoc Outdated

Comment thread cumulus/parachains/runtimes/assets/asset-hub-westend/src/lib.rs Outdated

Merge branch 'master' of github.com:paritytech/polkadot-sdk into fix-…

18d7acf

…ah-westend-local-starting

iulianbarbu commented Jul 15, 2025

View reviewed changes

Comment thread cumulus/parachains/runtimes/assets/asset-hub-westend/src/lib.rs Outdated

Comment thread prdoc/pr_9201.prdoc Outdated

Comment thread prdoc/pr_9201.prdoc Outdated

Apply suggestions from code review

ed6758b

iulianbarbu commented Jul 15, 2025

View reviewed changes

Comment thread prdoc/pr_9201.prdoc Outdated

Update prdoc/pr_9201.prdoc

0eba8c1

iulianbarbu requested a review from a team July 16, 2025 07:36

iulianbarbu commented Jul 16, 2025

View reviewed changes

Comment thread prdoc/pr_9201.prdoc Outdated

Update prdoc/pr_9201.prdoc

f79eae8

iulianbarbu added I5-enhancement An additional feature request. and removed I2-bug The node fails to follow expected behavior. labels Jul 16, 2025

iulianbarbu commented Jul 16, 2025

View reviewed changes

Comment thread prdoc/pr_9201.prdoc Outdated

Update prdoc/pr_9201.prdoc

f3a286c

skunert reviewed Jul 18, 2025

View reviewed changes

Comment thread cumulus/polkadot-omni-node/lib/src/common/spec.rs Outdated

skunert reviewed Jul 18, 2025

View reviewed changes

Comment thread prdoc/pr_9201.prdoc Outdated

skunert approved these changes Jul 18, 2025

View reviewed changes

Update pr_9201.prdoc

cc19091

Co-authored-by: Sebastian Kunert <mail@skunert.dev>

sistemd approved these changes Jul 29, 2025

View reviewed changes

remove para_id deprecation from logs/docs/code

22354df

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

iulianbarbu and others added 3 commits August 20, 2025 12:07

remove extra para_id params

6d3949c

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

fix clippy

63d3097

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>

Merge branch 'master' into fix-ah-westend-local-starting

a380e04

iulianbarbu mentioned this pull request Aug 20, 2025

[DRAFT] fixes and changes related to GetParachainInfo #9531

Closed

Merge branch 'master' into fix-ah-westend-local-starting

165494a

This was referenced Aug 21, 2025

Fix zombienet-substrate-0003-block-building-warp-sync #8868

Closed

Added GetParachainInfo runtime api polkadot-fellows/runtimes#860

Merged

sistemd approved these changes Aug 22, 2025

View reviewed changes

iulianbarbu added this pull request to the merge queue Aug 22, 2025

iulianbarbu added A4-backport-stable2409 A4-backport-unstable2507 Pull request must be backported to the unstable2507 release branch labels Aug 22, 2025

Merged via the queue into paritytech:master with commit 2660bf5 Aug 22, 2025
330 of 339 checks passed

iulianbarbu deleted the fix-ah-westend-local-starting branch August 22, 2025 13:40

paritytech-release-backport-bot Bot mentioned this pull request Aug 22, 2025

[stable2409] Backport #9201 #9541

Closed

paritytech-release-backport-bot Bot mentioned this pull request Aug 22, 2025

[unstable2507] Backport #9201 #9542

Closed

iulianbarbu removed the A4-backport-stable2409 label Aug 22, 2025

iulianbarbu mentioned this pull request Aug 22, 2025

polkadot-omni-node: update sections where stable2512 brings para_id removal polkadot-developers/polkadot-docs#795

Closed

pgherveou mentioned this pull request Aug 26, 2025

[pallet-revive] Update genesis config #9557

Merged

pgherveou added this to [preview] release tracker Mar 20, 2026

github-project-automation Bot moved this to Todo in [preview] release tracker Mar 20, 2026

github-project-automation Bot moved this from Todo to Done in [preview] release tracker Mar 20, 2026

pgherveou removed this from [preview] release tracker Mar 20, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

`polkadot-omni-node`: fixes and changes related to `GetParachainInfo`#9201

`polkadot-omni-node`: fixes and changes related to `GetParachainInfo`#9201
iulianbarbu merged 39 commits intoparitytech:masterfrom
iulianbarbu:fix-ah-westend-local-starting

iulianbarbu commented Jul 14, 2025 •

edited

Loading

Uh oh!

iulianbarbu commented Jul 14, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

paritytech-release-backport-bot Bot commented Aug 22, 2025

Uh oh!

paritytech-release-backport-bot Bot commented Aug 22, 2025

Uh oh!

Pray4Love1 commented Aug 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Conversation

iulianbarbu commented Jul 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Integration

Review Notes

Uh oh!

iulianbarbu commented Jul 14, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

paritytech-release-backport-bot Bot commented Aug 22, 2025

Uh oh!

paritytech-release-backport-bot Bot commented Aug 22, 2025

Uh oh!

Pray4Love1 commented Aug 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

iulianbarbu commented Jul 14, 2025 •

edited

Loading