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

Remove collation-generation subsystem from validator nodes #6832

Merged
merged 158 commits into from
Dec 12, 2024

Conversation

ndkazu
Copy link
Contributor

@ndkazu ndkazu commented Dec 10, 2024

Description

Issue #6476
Collation-generation is not needed for validators node, and should be removed.

Implementation

Use a DummySubsystem for collation_generation

@ndkazu
Copy link
Contributor Author

ndkazu commented Dec 10, 2024

Label needed please 🙏

@alindima alindima added the T8-polkadot This PR/Issue is related to/affects the Polkadot network. label Dec 11, 2024
@alindima
Copy link
Contributor

Set the blocking attribute on the collation_generation subsystem

why did you choose to do this?

@ndkazu
Copy link
Contributor Author

ndkazu commented Dec 11, 2024

Set the blocking attribute on the collation_generation subsystem

why did you choose to do this?

I am still trying to figure out how to use the orchestra_proc_macro, crate, and I just realized that I misunderstood the use of the blocking attribute.

@ndkazu
Copy link
Contributor Author

ndkazu commented Dec 11, 2024

@alindima , thank you for the hint!!! this time all the tests are good. I also did the change for validator_with_parallel_overseer_builder.

Copy link
Contributor

@alindima alindima left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@alindima alindima added this pull request to the merge queue Dec 12, 2024
@alindima alindima linked an issue Dec 12, 2024 that may be closed by this pull request
Merged via the queue into paritytech:master with commit 50e5dd2 Dec 12, 2024
201 of 202 checks passed
dudo50 pushed a commit to paraspell-research/polkadot-sdk that referenced this pull request Jan 4, 2025
…h#6832)

# Description
Issue paritytech#6476 
Collation-generation is not needed for validators node, and should be
removed.

## Implementation
Use a `DummySubsystem` for `collation_generation`

---------

Co-authored-by: Bastian Köcher <[email protected]>
Co-authored-by: command-bot <>
Co-authored-by: Dmitry Markin <[email protected]>
Co-authored-by: Alexandru Vasile <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T8-polkadot This PR/Issue is related to/affects the Polkadot network.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove collation-generation subsystem from validator nodes
6 participants