Skip to content
Snippets Groups Projects

Upgrade polkadot v1.17.0

Merged Upgrade polkadot v1.17.0
upgrade-polkadot-v1.17.0 into master
1 unresolved thread
Merged Benjamin Gallois requested to merge upgrade-polkadot-v1.17.0 into master
1 unresolved thread
  • Upgrade to 1.17.0 (see upgrade notes).
  • Add a benchmark for revoke_identity_legacy.
  • Add a build test to check autogenerated weights.
  • Apply several autofixes for Polkadot-SDK 1.17.0.
  • Autoformatting failing.
  • Move runtime benchmarks to common runtime.
  • Move offchain traits to common runtime.

Note for later (or not):

I left the frame_system_offchain traits implementation in separate chains. Maybe it would be interesting to add another macro and move a large part of each chain’s repetitive code (mainly the offchain traits, mod benches, and the TxExtensions) that is exactly the same and will remain the same to the common runtime. This would make it easier to maintain, as there would be no need to reflect changes in each separate runtime every time.

Release Note:

  • Introduced a new transaction extensions mechanism.
  • Reimplemented the nonce check in new extensions.
  • Resolved issue #6127: Ignore warnings from non-stable Rust.
  • New transaction_pool API.
  • New benchmarks and weights for pallet_transaction_payment.
  • New Inherent and offchain APIs for the runtime.
Edited by Benjamin Gallois

Merge request reports

Merge request pipeline #39804 passed

Stage: labels
Stage: quality
Stage: build
Stage: tests
Stage: deploy

Merge request pipeline passed for ebafb3c3

Merged by Hugo TrentesauxHugo Trentesaux 4 months ago (Feb 4, 2025 8:04pm UTC)

Loading

Pipeline #39807 passed

Stage: build
Stage: tests
Stage: deploy

Pipeline passed for cf605c4e on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Hugo Trentesaux
    Hugo Trentesaux @HugoTrentesaux
    Owner

    It is difficult to review this without reviewing substrate itself! There are a lot of API changes. Thanks for digging this :pray:

    I'm adding @tuxmain as reviewer for the new tx extension for oneshot account.

    I wonder what will break when we perform the upgrade of client and runtime on gdev network :thinking:. Do you have an idea?

  • Hugo Trentesaux approved this merge request

    approved this merge request

  • Hugo Trentesaux requested review from @tuxmain and removed review request for @HugoTrentesaux

    requested review from @tuxmain and removed review request for @HugoTrentesaux

  • Pascal Engélibert approved this merge request

    approved this merge request

  • Benjamin Gallois mentioned in merge request !314

    mentioned in merge request !314

  • Benjamin Gallois mentioned in issue #285 (closed)

    mentioned in issue #285 (closed)

  • Benjamin Gallois added 1 commit

    added 1 commit

    Compare with previous version

  • Hugo Trentesaux merged

    merged

  • Benjamin Gallois mentioned in commit cf605c4e

    mentioned in commit cf605c4e

    • Please register or sign in to reply