Skip to content

Added GetParachainInfo runtime api#860

Merged
fellowship-merge-bot[bot] merged 3 commits intopolkadot-fellows:mainfrom
bkontur:bko-get-para-info
Aug 22, 2025
Merged

Added GetParachainInfo runtime api#860
fellowship-merge-bot[bot] merged 3 commits intopolkadot-fellows:mainfrom
bkontur:bko-get-para-info

Conversation

@bkontur
Copy link
Copy Markdown
Contributor

@bkontur bkontur commented Aug 21, 2025

Not sure if this is really needed now, please, give here more context @iulianbarbu

  • Does not require a CHANGELOG entry

@bkontur
Copy link
Copy Markdown
Contributor Author

bkontur commented Aug 21, 2025

/cmd fmt

Copy link
Copy Markdown
Contributor

@acatangiu acatangiu left a comment

Choose a reason for hiding this comment

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

It def doesn't hurt having them even if not mandatory

@acatangiu acatangiu mentioned this pull request Aug 21, 2025
@iulianbarbu
Copy link
Copy Markdown

iulianbarbu commented Aug 21, 2025

The API should be added to simplify chain specs (by removing the para_id field from there). Otherwise, as long as we're using stable2506-X node to run the runtimes (not master or unstable2507 yet), we should be fine with or without these APIs.

All in all it doesn't hurt to merge them. stable2509 nodes will be able to use the new runtime API, so chain specs can omit the para_id at that time, if nodes are started from a state where runtime already contains the GetParachainInfo API (e.g. after the runtime upgrade).

I will backport paritytech/polkadot-sdk/#9201 to polkadot-sdk/unstable2507 to have a node that supports runtimes without GetParachainInfo (since unstable2507/current polkadot-sdk master contains a not fully functional node as it seems, and also the runtime API in cumulus_primitives_core), but I also need to test it a bit before that.

@github-actions
Copy link
Copy Markdown

Command "fmt" has started 🚀 See logs here

@github-actions
Copy link
Copy Markdown

Command "fmt" has finished ✅ See logs here

@acatangiu
Copy link
Copy Markdown
Contributor

/merge

@fellowship-merge-bot
Copy link
Copy Markdown
Contributor

Enabled auto-merge in Pull Request

Available commands
  • /merge: Enables auto-merge for Pull Request
  • /merge cancel: Cancels auto-merge for Pull Request
  • /merge help: Shows this menu

For more information see the documentation

@fellowship-merge-bot fellowship-merge-bot Bot enabled auto-merge (squash) August 22, 2025 07:26
@fellowship-merge-bot fellowship-merge-bot Bot merged commit 79f5003 into polkadot-fellows:main Aug 22, 2025
54 of 61 checks passed
@bkontur bkontur deleted the bko-get-para-info branch August 22, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants