Skip to content

Check that selected node sets fulfill the desired replication properties #3571

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

Closed
wants to merge 1 commit into from

Conversation

tillrohrmann
Copy link
Contributor

This commit adds a check that verifies that a selected node set fulfills the desired replication property before reconfiguring a log. This ensures that we don't end up in a situation where a log becomes unwritable because there is no chance to find a write quorum in the node set.

This fixes #3564.

Copy link
Contributor

@AhmedSoliman AhmedSoliman left a comment

Choose a reason for hiding this comment

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

This has been reviewed as part of the other PR, it's good to go, but I'd recommend we treat the failure here as non-retryable unless upper layers don't handle that path well.

This commit adds a check that verifies that a selected node set fulfills the
desired replication property before reconfiguring a log. This ensures that we
don't end up in a situation where a log becomes unwritable because there is
no chance to find a write quorum in the node set.

This fixes restatedev#3564.
@tillrohrmann
Copy link
Contributor Author

Merged via 221af91

@github-actions github-actions bot locked and limited conversation to collaborators Jul 24, 2025
@tillrohrmann tillrohrmann deleted the issues/3564 branch July 24, 2025 15:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Log auto improvement can select node sets which don't fulfill the replication property
2 participants