Skip to content

Conversation

@pandres95
Copy link
Member

@pandres95 pandres95 commented Feb 25, 2025

This Pull Request provides a minimal set of changes to support the upgrade to stable2412 (to the latest available patch).

Notable changes

Here are the most notable changes:

  • sp-std removed for every runtime (except Encointer). Now we use core or alloc instead.
  • Mocked weights for frame_system_extensions, pallet_transaction_payment, and xcm on most runtimes.
  • Mocked weights for pallet_identity on People runtimes.
  • Update to XCMv5 started.

Notable TODOs

  • Review whether we need to make more changes regarding XCMv5 (moved to a follow-up PR). @franciscoaguirre
  • In Asset Hubs, the benchmarking options for pallet-asset-conversion-tx-payment is not exposed (or at least, is raising errors when trying to add it).
  • For polkadot-runtime, do we still need coretime_migration.rs? @seadanda
  • Assist with the initial implementation of TransactionExtension @georgepisaltu @gui1117

Open Question

  • Shouldn't we start using frame, #[frame_construct_runtime] and #[derive_impl] since they've already been proven to work?

Help needed

  • @clangenb to update and re-enable Encointer.

Action List

  • Remove any reference to either NetworkId::Westend and NetworkId::Rococo and replace with NetworkId::ByGenesis (moved to a follow-up PR).
  • Upgrade to stable2412-3
  • Setup filter for pallet-staking to exclude pool members. See comment here.

Follow-up PRs

  • Refresh weights (to do before this release). ;)
  • Move storage for ForeignAssets and AssetConversion to XCMv5.
  • Remove sp-std.

Note: This had to be done since there are not updated pallets that could be used for upgrading the runtime.
TODO @clangenb: Help updating Encointer to `stable2412`.
Note: includes mocked weights, needs to run proper benches.
Note 2: A migration to XCMv5 was started. Do we need anything additional?
Note 3: `sp_std` was removed.
Note: includes mocked weights, needs to run proper benches.
Note 2: A migration to XCMv5 was started. Do we need anything additional?
Note 3: `sp_std` was removed.
Note 4: do we still need `coretime_migration.rs`?
Note: includes mocked weights, needs to run proper benches.
Note 2: A migration to XCMv5 was started. Do we need anything additional?
Note 3: `sp_std` was removed.
Note: includes mocked weights, needs to run proper benches.
Note 2: A migration to XCMv5 was started. Do we need anything additional?
Note 3: `sp_std` was removed.
Note: includes mocked weights, needs to run proper benches.
Note 2: `sp_std` was removed
Note: includes mocked weights, needs to run proper benches.
Note 2: `sp_std` was removed
Note: includes mocked weights, needs to run proper benches.
Note 2: `sp_std` was removed
Note: includes mocked weights, needs to run proper benches.
Note 2: A migration to XCMv5 was started. Do we need anything additional?
Note 3: `sp_std` was removed
Note: includes mocked weights, needs to run proper benches.
Note 2: A migration to XCMv5 was started. Do we need anything additional?
Note 3: `sp_std` was removed
Note: includes mocked weights, needs to run proper benches.
Note 3: `sp_std` was removed
Note: includes mocked weights, needs to run proper benches.
Note 3: `sp_std` was removed
Note: includes mocked weights, needs to run proper benches.
Note 2: `sp_std` was removed
@pandres95 pandres95 changed the title Upgrade polkadot sdk to stable2412 Upgrade polkadot sdk to stable2412-1 Feb 25, 2025
@clangenb
Copy link
Contributor

@pandres95 on-it. I reckon to have a PR by Thursday, the latest.

# Conflicts:
#	relay/kusama/src/lib.rs
#	relay/polkadot/src/lib.rs
#	system-parachains/asset-hubs/asset-hub-kusama/src/lib.rs
#	system-parachains/asset-hubs/asset-hub-polkadot/src/lib.rs
#	system-parachains/bridge-hubs/bridge-hub-kusama/src/lib.rs
#	system-parachains/bridge-hubs/bridge-hub-polkadot/src/lib.rs
#	system-parachains/collectives/collectives-polkadot/src/lib.rs
#	system-parachains/coretime/coretime-kusama/src/lib.rs
#	system-parachains/coretime/coretime-polkadot/src/lib.rs
#	system-parachains/gluttons/glutton-kusama/src/lib.rs
#	system-parachains/people/people-kusama/src/lib.rs
#	system-parachains/people/people-polkadot/src/lib.rs
@clangenb
Copy link
Contributor

encointer is ready: pandres95#9

* Revert "feat(encointer-kusama-runtime): disable Encointer runtime and pallets."

This reverts commit 9303cb3

* bump encointer crates

* migrate encointer runtime to stable202412-2

* [encointer] add benchmarking setup for asset_tx_payment

* [encointer] fix asset-tx-benchmarks

* fmt

* remove sp-std from encointer

* fix pallet-transaction payment feature propagation
@seadanda
Copy link
Contributor

Closes #552

Polkadot migration cleanup: pandres95#10

@bkontur
Copy link
Contributor

bkontur commented Apr 16, 2025

@pepoviola zombienet-smoke test does not start progressing:
https://github.com/polkadot-fellows/runtimes/actions/runs/14489755127/job/40643253645?pr=606#step:10:459

test smoke has been running for over 60 seconds
Error: The action 'Zombienet smoke test' has timed out after 20 minutes.

It is nothing like before: https://github.com/polkadot-fellows/runtimes/actions/runs/14001583911/job/39208932124

running 1 test
2025-03-21T22:14:27.629319Z  INFO smoke: Using zombienet provider: Docker

Could there something specific with stable2412 for zombienet?

_descendant_origin: &Option<InteriorLocation>,
_xcm: &Xcm<RuntimeCall>,
) -> Weight {
XcmGeneric::<Runtime>::execute_with_origin()
Copy link
Contributor

Choose a reason for hiding this comment

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

@franciscoaguirre shouldn't we disable this everywhere? or remove execute_with_origin?

Copy link
Contributor

Choose a reason for hiding this comment

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

We removed it from the executor anyway. If someone tries to use it it'll just return an error

Copy link
Contributor

Choose a reason for hiding this comment

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

aha, ok, I see that patch is here: so the benchmarking will return Weight::MAX right?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes

franciscoaguirre and others added 2 commits April 16, 2025 14:01
* fix(polkadot-runtime): pallet-xcm-benchmarks

* fix(polkadot-runtime): increate worst case holding
@bkontur
Copy link
Contributor

bkontur commented Apr 16, 2025

@pepoviola zombienet-smoke test does not start progressing: https://github.com/polkadot-fellows/runtimes/actions/runs/14489755127/job/40643253645?pr=606#step:10:459

test smoke has been running for over 60 seconds
Error: The action 'Zombienet smoke test' has timed out after 20 minutes.

It is nothing like before: https://github.com/polkadot-fellows/runtimes/actions/runs/14001583911/job/39208932124

running 1 test
2025-03-21T22:14:27.629319Z  INFO smoke: Using zombienet provider: Docker

Could there something specific with stable2412 for zombienet?

Trying some nits here for zombienet smoke: pandres95#24

@bkontur
Copy link
Contributor

bkontur commented Apr 16, 2025

@pepoviola zombienet-smoke test does not start progressing: https://github.com/polkadot-fellows/runtimes/actions/runs/14489755127/job/40643253645?pr=606#step:10:459

test smoke has been running for over 60 seconds
Error: The action 'Zombienet smoke test' has timed out after 20 minutes.

It is nothing like before: https://github.com/polkadot-fellows/runtimes/actions/runs/14001583911/job/39208932124

running 1 test
2025-03-21T22:14:27.629319Z  INFO smoke: Using zombienet provider: Docker

Could there something specific with stable2412 for zombienet?

Trying some nits here for zombienet smoke: pandres95#24

@pepoviola I don't know why, but zombienet smoke test is flaky, I tried coretime-polkadot and previous version with some cumulus based parachain, and it does not produce blocks for parachain:
https://github.com/pandres95/runtimes/actions/runs/14496644330/job/40666016974?pr=24

[2025-04-16T15:52:56Z INFO  smoke] Block #8
[2025-04-16T15:53:04Z INFO  smoke] Block #9
[2025-04-16T15:53:04Z INFO  smoke] Parachain Block #0
Error: The action 'Zombienet smoke test' has timed out after 20 minutes.

Co-authored-by: Oliver Tale-Yazdi <[email protected]>
@pandres95
Copy link
Member Author

/merge

@fellowship-merge-bot
Copy link
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) April 16, 2025 18:41
@fellowship-merge-bot fellowship-merge-bot bot merged commit aec5376 into polkadot-fellows:main Apr 16, 2025
58 of 59 checks passed
@pandres95 pandres95 changed the title Upgrade polkadot sdk to stable2412-4 Upgrade polkadot sdk to stable2412-4 (rel. 1.5.0) Apr 16, 2025
@pepoviola
Copy link
Contributor

@pepoviola zombienet-smoke test does not start progressing: https://github.com/polkadot-fellows/runtimes/actions/runs/14489755127/job/40643253645?pr=606#step:10:459

test smoke has been running for over 60 seconds
Error: The action 'Zombienet smoke test' has timed out after 20 minutes.

It is nothing like before: https://github.com/polkadot-fellows/runtimes/actions/runs/14001583911/job/39208932124

running 1 test
2025-03-21T22:14:27.629319Z  INFO smoke: Using zombienet provider: Docker

Could there something specific with stable2412 for zombienet?

Trying some nits here for zombienet smoke: pandres95#24

@pepoviola I don't know why, but zombienet smoke test is flaky, I tried coretime-polkadot and previous version with some cumulus based parachain, and it does not produce blocks for parachain: https://github.com/pandres95/runtimes/actions/runs/14496644330/job/40666016974?pr=24

[2025-04-16T15:52:56Z INFO  smoke] Block #8
[2025-04-16T15:53:04Z INFO  smoke] Block #9
[2025-04-16T15:53:04Z INFO  smoke] Parachain Block #0
Error: The action 'Zombienet smoke test' has timed out after 20 minutes.

That weird, can we increase the log (zombienet=debug) to get a more verbose output. I also will try locally.
Thx!

@bkontur
Copy link
Contributor

bkontur commented Apr 17, 2025

@pepoviola zombienet-smoke test does not start progressing: https://github.com/polkadot-fellows/runtimes/actions/runs/14489755127/job/40643253645?pr=606#step:10:459

test smoke has been running for over 60 seconds
Error: The action 'Zombienet smoke test' has timed out after 20 minutes.

It is nothing like before: https://github.com/polkadot-fellows/runtimes/actions/runs/14001583911/job/39208932124

running 1 test
2025-03-21T22:14:27.629319Z  INFO smoke: Using zombienet provider: Docker

Could there something specific with stable2412 for zombienet?

Trying some nits here for zombienet smoke: pandres95#24

@pepoviola I don't know why, but zombienet smoke test is flaky, I tried coretime-polkadot and previous version with some cumulus based parachain, and it does not produce blocks for parachain: https://github.com/pandres95/runtimes/actions/runs/14496644330/job/40666016974?pr=24

[2025-04-16T15:52:56Z INFO  smoke] Block #8
[2025-04-16T15:53:04Z INFO  smoke] Block #9
[2025-04-16T15:53:04Z INFO  smoke] Parachain Block #0
Error: The action 'Zombienet smoke test' has timed out after 20 minutes.

That weird, can we increase the log (zombienet=debug) to get a more verbose output. I also will try locally. Thx!

@pepoviola I changed the logs, but let's please continue discussion/fixing here: #676

@pandres95 pandres95 deleted the upgrade-polkadot-sdk-to-stable2412 branch May 6, 2025 17:52
ggwpez pushed a commit that referenced this pull request Jun 17, 2025
This is leftover from bump stable2412 where we added Aliasers but not
changed weights:
#606
for AssetHub weights were fixed here
#700 but not for other
chains

- [X] Does not require a CHANGELOG entry
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.