arup-chauhan opened a new pull request, #4646:
URL: https://github.com/apache/cassandra/pull/4646
## Summary
This PR adds regression coverage for CASSANDRA-20910 to ensure foreign
nodes are rejected during gossip SYN handling and cannot be admitted into
membership state and is a companion PR to the existing #4618
## What This Adds
- New test class: `org.apache.cassandra.gms.GossipDigestSynVerbHandlerTest`
- `rejectsForeignClusterNameWithoutAdmission`
- `rejectsForeignPartitionerWithoutAdmission`
## Invariants Covered
- A node with mismatched cluster identity must not be admitted.
- A node with mismatched partitioner must not be admitted.
## Assertions
For both mismatch cases, tests verify:
- mismatch warning is logged (`ClusterName mismatch` / `Partitioner
mismatch`)
- gossip message is not processed forward (`lastProcessedMessageAt`
unchanged)
- endpoint is not added to membership state (`endpointStateMap` /
`liveEndpoints`)
## Test Plan
```bash
JDK11_HOME=$(jenv prefix 11.0.30)
JAVA_HOME="$JDK11_HOME" PATH="$JAVA_HOME/bin:$PATH" ant -Duse.jdk11=true
testsome -Dtest.name=org.apache.cassandra.gms.GossipDigestSynVerbHandlerTest
JDK11_HOME=$(jenv prefix 11.0.30)
JAVA_HOME="$JDK11_HOME" PATH="$JAVA_HOME/bin:$PATH" ant -Duse.jdk11=true
testsome -Dtest.name=org.apache.cassandra.gms.GossiperTest
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
