jsancio commented on PR #15986: URL: https://github.com/apache/kafka/pull/15986#issuecomment-2145613112
> I don't think we want to change all of the tests to use `controller.quorum.bootstrap.servers`. We still need to support the old configuration. Let's make it so that tests using IBP_3_8_IV0 or newer use the new thing, and tests using an older MV use the old configuration. That way we will have good coverage. `controller.quorum.bootstrap.servers` is not gated by metadata version, kraft version or IBP. This property only affects the behavior of the local node and it doesn't affect the network protocol (RPCs) or data version stored on disk. In other words using this configuration should be safe irrespective of the software version of the remote nodes. I decided to change all of the quorum test harness test to use `controller.quorum.bootstrap.servers` because this feature is a super set of the `controller.quorum.voters` feature since the id are negatives. In other words, if tests pass with `controller.quorum.bootstrap.servers` they should pass with `controller.quorum.bootstrap.servers`. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
