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

WIP: Fix coretime partioning + super low latency on-demand #6224

Draft
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

eskimor
Copy link
Member

@eskimor eskimor commented Oct 24, 2024

TODOS:

  • Make it typecheck
  • Make CoreState one state item for all cores
  • Runtime APIs must look into the future (retrieve claim queue for now +1)
  • Update scheduler + paras_inherent to use the new pop all cores at once API
  • Fix scheduler and paras_inherent to use the new API
  • Add back duplication of first assignment for on-demand
  • Limit assignments returned by the scheduler to availability cores count
  • Fix peek for on-demand
  • Fix benchmarks
  • Migrations
  • Add test that peek at block x correctly predicts pops at block x + n
  • Add test that on-demand order on empty core shows up in two next blocks (and no more)
  • Add test that bulk assignment coming in on an unoccupied core gets duplicated before the start
  • Add test that peek does not modify state (multiple calls will yield the same result)
  • Fix tests
  • Update docs
  • Drop other parachain assigner
  • Move coretime & ondemand assigner into scheduler folder
  • Polish

Fixes: #1312 #6563

eskimor added 6 commits October 24, 2024 18:13
@eskimor eskimor changed the title WIP: Introduce peeking for the AssignmentProvider. WIP: Fix coretime partioning + super low latency on-demand Oct 25, 2024
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/12007919486
Failed job name: build-runtimes-polkavm

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.

Instant on-demand orders
1 participant