Skip to content

Remove MaxSessionKeysLength and MaxSessionKeysProofLength#11115

Merged
bkchr merged 7 commits intomasterfrom
cirko33-issue11083
Feb 19, 2026
Merged

Remove MaxSessionKeysLength and MaxSessionKeysProofLength#11115
bkchr merged 7 commits intomasterfrom
cirko33-issue11083

Conversation

@cirko33
Copy link
Copy Markdown
Member

@cirko33 cirko33 commented Feb 19, 2026

fixes the issue #11083 where MaxSessionKeysLength and MaxSessionKeysProofLength were unnecessary because there are not stored, just validated.

@cirko33 cirko33 self-assigned this Feb 19, 2026
@cirko33 cirko33 requested a review from a team as a code owner February 19, 2026 12:45
@cirko33 cirko33 added the A4-backport-stable2603 Pull request must be backported to the stable2603 release branch label Feb 19, 2026
@cirko33
Copy link
Copy Markdown
Member Author

cirko33 commented Feb 19, 2026

/cmd prdoc --audience runtime_dev --bump patch

Comment thread substrate/frame/staking-async/integration-tests/src/ah/test.rs Outdated
@sigurpol
Copy link
Copy Markdown
Contributor

sigurpol commented Feb 19, 2026

LGTM - just remove the BoundedVec import from ah/test.rs and CI should be happier.
Plus you need to update westend AH and staking-async runtimes to nuke the MaxSessionKeys* stuff from there too. Same where we do benchmarking e.g. substrate/frame/staking-async/rc-client/src/benchmarking.rs#L70 (just grep for all occurrences of these 2 bad boys and 🪓🪓🪓)

(and don't forget to update the prdoc too to add the corresponding crates)

@cirko33
Copy link
Copy Markdown
Member Author

cirko33 commented Feb 19, 2026

/cmd prdoc --force --audience runtime_dev --bump patch

Comment thread prdoc/pr_11115.prdoc Outdated
@cirko33
Copy link
Copy Markdown
Member Author

cirko33 commented Feb 19, 2026

/cmd prdoc --force --audience runtime_dev --bump patch

@sigurpol sigurpol added the T2-pallets This PR/Issue is related to a particular pallet. label Feb 19, 2026
@bkchr bkchr enabled auto-merge February 19, 2026 21:53
@bkchr bkchr added this pull request to the merge queue Feb 19, 2026
Merged via the queue into master with commit 5049801 Feb 19, 2026
241 of 246 checks passed
@bkchr bkchr deleted the cirko33-issue11083 branch February 19, 2026 22:41
paritytech-release-backport-bot Bot pushed a commit that referenced this pull request Feb 19, 2026
fixes the issue #11083 where MaxSessionKeysLength and
MaxSessionKeysProofLength were unnecessary because there are not stored,
just validated.

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 5049801)
@paritytech-release-backport-bot
Copy link
Copy Markdown

Successfully created backport PR for stable2603:

EgorPopelyaev added a commit that referenced this pull request Feb 20, 2026
Backport #11115 into `stable2603` from cirko33.

See the
[documentation](https://github.com/paritytech/polkadot-sdk/blob/master/docs/BACKPORT.md)
on how to use this bot.

<!--
  # To be used by other automation, do not modify:
  original-pr-number: #${pull_number}
-->

---------

Co-authored-by: Luka Ciric <luka.ciric2106@gmail.com>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Egor_P <egor@parity.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A4-backport-stable2603 Pull request must be backported to the stable2603 release branch T2-pallets This PR/Issue is related to a particular pallet.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants