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

backport to master: Handling of disabled validators in backing subsystem (#1259) #2764

Merged
merged 5 commits into from
Jan 8, 2024

Conversation

ordian
Copy link
Member

@ordian ordian commented Dec 20, 2023

#1259 was merged into a feature branch, but we've decided to merge node-side changes for disabling straight into master.
This is a dependency of #1841 and #2637.

Handles validator disabling in the backing subsystem. The PR introduces
two changes:
1. Don't import statements from disabled validators.
2. Don't validate and second if the local validator is disabled.

The purpose of this change is first to ignore statements from disabled
validators on the node side. This is an optimisation as these statements
are supposed to be cleared in the runtime too (still not implemented at
the moment). Additionally if the local node is a validator which is
disabled - don't process any new candidates.

---------

Co-authored-by: ordian <noreply@reusable.software>
Co-authored-by: ordian <write@reusable.software>
@ordian ordian added the T8-polkadot This PR/Issue is related to/affects the Polkadot network. label Dec 20, 2023
@ordian ordian marked this pull request as ready for review December 20, 2023 15:22
@tdimitrov tdimitrov requested a review from Overkillus January 5, 2024 12:22
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.

Respect Disablement in Backing
3 participants