Skip to content

Add --experimental-collator-protocol cli argument to enable the new collator protocol implementation#10285

Merged
serban300 merged 2 commits intoalindima/collator-protocol-revamp-collation-managerfrom
tsv-collator-proto-revamp-cli-arg
Nov 11, 2025
Merged

Add --experimental-collator-protocol cli argument to enable the new collator protocol implementation#10285
serban300 merged 2 commits intoalindima/collator-protocol-revamp-collation-managerfrom
tsv-collator-proto-revamp-cli-arg

Conversation

@tdimitrov
Copy link
Copy Markdown
Contributor

No description provided.

@tdimitrov tdimitrov requested a review from serban300 November 11, 2025 11:26
@serban300 serban300 merged commit 11a9474 into alindima/collator-protocol-revamp-collation-manager Nov 11, 2025
93 of 168 checks passed
@serban300 serban300 deleted the tsv-collator-proto-revamp-cli-arg branch November 11, 2025 13:49
@paritytech-workflow-stopper
Copy link
Copy Markdown

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

github-merge-queue bot pushed a commit that referenced this pull request Feb 4, 2026
Implements the `CollationManager` and the new collator protocol
(validator side) subsystem.

Issues #8182 and
#7752.

These are the big remaining parts which would enable us to test the
entire implementation.

TODO:
- [ ] add a couple more unit tests (see the suggestions at the bottom of
the tests file)
- [x] polish the ClaimQueueState and verify if it's sufficiently covered
by unit tests
  - #10334
  - #10368
- [x] add metrics and polish logs -
#10730
- [x] add a CLI parameter for enabling the experimental subsystem (and
remove the compile-time feature) ->
#10285
- [x] implement registered paras update, using
#9055
- [ ] do some manual zombienet tests with v1 protocol version and with
restarting validators (including syncing with warp sync)
- [x] prdoc
- [x] Rollback 
-
03e8915
-
05e1497
These commits were added just to run the CI tests for this PR with the
new experimental protocol

After merging: 
- [ ] versi testing



Uses a slightly modified version of the ClaimQueueState written by
@tdimitrov in #7114.

---------

Co-authored-by: Tsvetomir Dimitrov <[email protected]>
Co-authored-by: Serban Iorga <[email protected]>
Co-authored-by: Serban Iorga <[email protected]>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants