Skip to content

Conversation

@manuelmauro
Copy link
Contributor

@manuelmauro manuelmauro commented Oct 8, 2025

⚠️ Breaking Changes ⚠️

  • Not a breaking change, but notice that DOT/KSM reserve calls should use transfer_assets_using_type_and_then for avoiding errors related to reserve misconfiguration.

What does it do?

Upgrades polkadot-sdk to stable2506.

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

@manuelmauro manuelmauro self-assigned this Oct 8, 2025
@manuelmauro manuelmauro added D9-needsaudit👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited breaking Needs to be mentioned in breaking changes labels Oct 8, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 8, 2025

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • agent-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch moonbeam-polkadot-stable2506

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@manuelmauro manuelmauro force-pushed the moonbeam-polkadot-stable2506 branch from b782ed8 to f44e21c Compare October 10, 2025 12:54
This adds the required OnNewHead configuration parameter introduced in paritytech/polkadot-sdk#8531
Upstream PR: Moonsong-Labs/moonkit#83 - "Use PostInherents to validate timestamp"   - Author: Rodrigo Quelhas (RomarQ)   - Merged: September 30, 2025    Details: This trait item is used in the PostInherents validation to ensure that the configured block time in the node and runtime is compatible with Nimbus's consensus mechanism.   The value is set to MILLISECS_PER_BLOCK (12000ms for Moonbeam), matching the parachain's block production time. This change prepares for Polkadot SDK updates that deprecated the   CheckInherents parameter in favor of PostInherents validation.
Upstream PR: paritytech/polkadot-sdk#8299 - "Allow building on older relay parents"   - Author: Sebastian Kunert (skunert)   - Merged: May 29, 2025    Details: With an offset of 0, blocks are built on the current relay chain tip. Non-zero values allow building on older relay parents (e.g., offset of 2 means building 2 blocks   behind the tip) to avoid building on short-lived relay chain forks. The runtime enforces this offset by requiring multiple relay parent descendants to be present in the   set_validation_data inherent.
…ntId>> bound

Fixed RuntimeOrigin trait bounds - Removed From<Option<AccountId>> bound and updated all precompiles to use frame_system::RawOrigin::Signed().into()
Added proper type qualifications and EncodeLike trait bound in crowdloan-rewards precompile
Copy link
Contributor

@RomarQ RomarQ left a comment

Choose a reason for hiding this comment

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

The benchmarks are failing: https://github.com/moonbeam-foundation/moonbeam/actions/runs/19515796274/job/55867083680

We should also update the breaking changes section and mention the important changes from polkadot-sdk

@RomarQ RomarQ merged commit 89eb3e5 into master Nov 26, 2025
53 checks passed
@RomarQ RomarQ deleted the moonbeam-polkadot-stable2506 branch November 26, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes breaking Needs to be mentioned in breaking changes D9-needsaudit👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants