Skip to content

feat: migrated pallet_throttle to UncheckedOnRuntimeUpgrade#1526

Merged
Klapeyron merged 1 commit into
mainfrom
feat/versioned_migration_for_pallet_throttle
May 19, 2026
Merged

feat: migrated pallet_throttle to UncheckedOnRuntimeUpgrade#1526
Klapeyron merged 1 commit into
mainfrom
feat/versioned_migration_for_pallet_throttle

Conversation

@Klapeyron

@Klapeyron Klapeyron commented May 18, 2026

Copy link
Copy Markdown
Contributor

Overview

Refactors the throttle pallet AccountUsage storage migration to follow the Polkadot SDK UncheckedOnRuntimeUpgrade + VersionedMigration<0, 1, ...> pattern. Keeps the existing cleanup behavior for legacy values encoded as (bytes_used, window_start) and adds regression coverage for that storage shape.

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • All commits are signed off (git commit -s) for the DCO
  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

@Klapeyron Klapeyron requested a review from a team as a code owner May 18, 2026 10:48
@Klapeyron Klapeyron self-assigned this May 18, 2026
@Klapeyron Klapeyron force-pushed the feat/versioned_migration_for_pallet_throttle branch from acc999a to 1415b79 Compare May 18, 2026 11:05
@Klapeyron Klapeyron linked an issue May 18, 2026 that may be closed by this pull request
LGLO
LGLO previously approved these changes May 18, 2026
LGLO
LGLO previously approved these changes May 18, 2026
Signed-off-by: Tomasz Bartos <tomasz.bartos@shielded.io>
@Klapeyron Klapeyron force-pushed the feat/versioned_migration_for_pallet_throttle branch from 4ae900c to 3159a4d Compare May 18, 2026 17:13
@Klapeyron Klapeyron enabled auto-merge May 18, 2026 18:18
@Klapeyron Klapeyron added this pull request to the merge queue May 19, 2026
Merged via the queue into main with commit 7700aba May 19, 2026
29 of 31 checks passed
@Klapeyron Klapeyron deleted the feat/versioned_migration_for_pallet_throttle branch May 19, 2026 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:node Node component priority:high P2 — current sprint

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: add versioned migration to pallet throttle

3 participants