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

slot-based-collator: Move spawning of the futures #6561

Merged
merged 4 commits into from
Nov 21, 2024
Merged

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Nov 20, 2024

Move spawning of the slot-based collator into the run function. Also the tasks are being spawned as blocking task and not just as normal tasks.

Move spawning of the slot-based collator into the `run` function. Also the tasks are being spawned as blocking task and not just as normal tasks.
@bkchr bkchr added the T9-cumulus This PR/Issue is related to cumulus. label Nov 20, 2024
@bkchr bkchr requested a review from a team November 20, 2024 16:08
@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/11937246140
Failed job name: test-linux-stable

@bkchr
Copy link
Member Author

bkchr commented Nov 20, 2024

/cmd prdoc --audience node_dev --bump major

@bkchr bkchr requested a review from a team November 21, 2024 08:00
@bkchr bkchr added this pull request to the merge queue Nov 21, 2024
Merged via the queue into master with commit 56d97c3 Nov 21, 2024
198 of 202 checks passed
@bkchr bkchr deleted the bkchr-slot-based-spawn branch November 21, 2024 11:04
Krayt78 pushed a commit to Krayt78/polkadot-sdk that referenced this pull request Dec 18, 2024
Move spawning of the slot-based collator into the `run` function. Also
the tasks are being spawned as blocking task and not just as normal
tasks.

---------

Co-authored-by: GitHub Action <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T9-cumulus This PR/Issue is related to cumulus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants