Skip to content

Update to polkadot-sdk stable2506 and Rust 1.88#1302

Merged
tmpolaczyk merged 131 commits intomasterfrom
tanssi-polkadot-stable2506
Nov 3, 2025
Merged

Update to polkadot-sdk stable2506 and Rust 1.88#1302
tmpolaczyk merged 131 commits intomasterfrom
tanssi-polkadot-stable2506

Conversation

@tmpolaczyk
Copy link
Copy Markdown
Contributor

@tmpolaczyk tmpolaczyk commented Sep 29, 2025

Please also review the snowbridge changes in our polkadot-sdk fork: moondance-labs/polkadot-sdk#15

Notable changes:

  • litep2p is the default now

libp2p can still be used by passing --network-backend=libp2p.

We include a few cherry-picks to fix litep2p issues, using litep2p 0.11.1. The official polkadot stable2506 release uses 0.9.5, so there may be some issues that affect polkadot relay chains but not tanssi. litep2p diff

This broke some of our tests, but it's something that should only happen in a local network when running multiple nodes in the same machine. See #1337

  • fork-aware-tx-pool is the default now

We were already using it almost everywhere so this is fine. I'm leaving the explicit tx-pool CLI args in zombienet tests for now, we can clean it up later.

We get a deprecation warning so we removed it from our pallets. But parity decided to allow(deprecated) so we still must include it for parity pallets.

To improve bootnode discoverability, doesn't affect us much because we use the data preservers pallet. Moved to a separate PR for easier review: #1344

Anti-DOS feature, we don't care.

  • state_at now takes TrieCacheContext arg. Onchain => Trusted, Offchain => Untrusted.

This is an optimization to promise that the cache size is bounded, we only use Untrusted in our codebase, copied from the state_at impl from cumulus.

Allows collators to build blocks using an older relay block. Disabled now (set to 0). It allows to reduce forks on the parachain side, but it increases the delay for processing XCM messages.

  • Some changes in collator code related to relay_parent_descendants.

This is related to the RelayParentOffset. We didn't enable that feature so this should simply be a refactor. I fixed some small discrepancies between our collator and upstream, and created a ticket to fix some more (MD-1403).

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 29, 2025

WASM runtime size check:

Compared to target branch

dancebox runtime: 1848 KB (+156 KB) ⚠️

flashbox runtime: 1080 KB (+36 KB) ⚠️

dancelight runtime: 2592 KB (+76 KB) 🚨

starlight runtime: 2552 KB (+80 KB) 🚨

container chain template simple runtime: 1452 KB (+104 KB) ⚠️

container chain template frontier runtime: 1816 KB (+168 KB) ⚠️

@tmpolaczyk tmpolaczyk added the A8-mergeoncegreen Pull request is reviewed well. label Sep 29, 2025
Comment thread test/utils/block.ts Outdated
Comment thread test/utils/block.ts Outdated
Comment thread chains/orchestrator-relays/runtime/dancelight/src/lib.rs
Comment thread chains/runtime-common/src/relay.rs
Comment thread client/consensus/src/collators/basic.rs
Comment thread client/service-container-chain-spawner/src/cli.rs
Comment thread chains/orchestrator-paras/runtime/dancebox/src/lib.rs
Comment thread client/consensus/src/collators.rs Outdated
Comment thread client/consensus/src/collators.rs
Comment thread chains/orchestrator-paras/runtime/dancebox/src/xcm_config.rs
Comment thread client/consensus/src/collators.rs
Copy link
Copy Markdown
Contributor

@evgeny-s evgeny-s left a comment

Choose a reason for hiding this comment

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

@tmpolaczyk
We have a job that checks if we need to update runtime API-s.
We can enable it using the label: dependency-upgrade.
Do you want to include it?
Or run it manually

Copy link
Copy Markdown
Contributor

@dimartiro dimartiro left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@nanocryk nanocryk left a comment

Choose a reason for hiding this comment

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

LGTM!

@tmpolaczyk tmpolaczyk enabled auto-merge (squash) November 3, 2025 12:20
@tmpolaczyk tmpolaczyk merged commit e5df510 into master Nov 3, 2025
216 of 220 checks passed
@tmpolaczyk tmpolaczyk deleted the tanssi-polkadot-stable2506 branch November 3, 2025 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A8-mergeoncegreen Pull request is reviewed well. 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 dependency-upgrade R9-allruntimes change affects all runtimes V-1600 to incorporate in runtime version 1600

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants