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

3.x: Add config for allowing zero-token nodes #400

Merged

Conversation

Bouncheck
Copy link
Collaborator

@Bouncheck Bouncheck commented Dec 13, 2024

Adds an option that controls if system.peers rows without tokens should
be considered valid. Enabling this option will allow discovering zero-token
peers with extended_peer_check enabled (it is enabled by default).

Adds ZeroTokenNodesIT that verifies the driver behavior. It also contains
some test methods relating to zero-token DCs. The special driver behavior for
arbiter DCs is not yet fully decided so those methods can change in the future.

@Bouncheck Bouncheck force-pushed the scylla-3.x-zero-token-ITs-v2 branch from e54498a to 25e4465 Compare December 13, 2024 11:09
@Bouncheck
Copy link
Collaborator Author

Made another version with different approach according to suggestions. @dkropachev we can choose which one to proceed with.

@Bouncheck Bouncheck force-pushed the scylla-3.x-zero-token-ITs-v2 branch 4 times, most recently from b49eca4 to de16bb0 Compare December 19, 2024 13:16
@Bouncheck Bouncheck changed the title 3.x: Adjust for zero token nodes (option B) 3.x: Add config for allowing zero-token nodes Dec 19, 2024
@Bouncheck Bouncheck force-pushed the scylla-3.x-zero-token-ITs-v2 branch from de16bb0 to 6458a1c Compare December 19, 2024 13:19
Adds an option that controls if `system.peers` rows without tokens should
be considered valid. Enabling this option will allow discovering zero-token
peers with extended_peer_check enabled (it is enabled by default).

Adds ZeroTokenNodesIT that verifies the driver behavior. It also contains
some test methods relating to zero-token DCs. The special driver behavior for
arbiter DCs is not yet fully decided so those methods can change in the future.
@dkropachev dkropachev merged commit f4de8e7 into scylladb:scylla-3.x Dec 23, 2024
11 checks passed
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