Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Coretime chain] Add high assignment count mitigation to testnets #6022

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

seadanda
Copy link
Contributor

Fixed in Polkadot and Kusama in polkadot-fellows/runtimes#434 and this PR just adds to testnets.

We can handle a maximum of 28 assignments inside one XCM, while it's possible to have 80 (if a region is interlaced 79 times).
This can be chunked on the coretime chain side but currently the scheduler on the relay makes assumptions that means we can't send more than one chunk for a given core.

This just truncates the additional assignments until we can extend the relay to support this. This means that the first 27 assignments are taken, the final 28th is used to pad with idle to complete the mask (the relay also assumes that every schedule is complete). Any other assignments are dropped.

@seadanda seadanda added the T14-system_parachains This PR/Issue is related to system parachains. label Oct 11, 2024
@seadanda
Copy link
Contributor Author

seadanda commented Oct 11, 2024

/cmd prdoc --pr 6022 --bump patch --audience "Runtime User"
edit: this should have been --audience "runtime_user"

@seadanda seadanda marked this pull request as ready for review October 16, 2024 12:34
Copy link
Member

@bkchr bkchr left a comment

Choose a reason for hiding this comment

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

Is the underlying problem tracked in an issue?

@seadanda
Copy link
Contributor Author

Just made it - that was too far down on my TODO list but sorted now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T14-system_parachains This PR/Issue is related to system parachains.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants