kevin-wu24 commented on PR #17581: URL: https://github.com/apache/kafka/pull/17581#issuecomment-2452632231
> @mumrah @cmccabe—is there an existing Jira that PR overlaps with the changes in this PR? I want to make sure I'm not duplicating efforts and/or stepping on anyone’s work. > > Ideally there would be two PRs: one for the integration tests and one for the system tests, as @mumrah suggested above. > > Thanks! Your system test changes are a superset of the changes I made in https://github.com/apache/kafka/pull/17638/files (involves just removing explicit `quorum.zk` parameterizations), which is tracked by https://issues.apache.org/jira/browse/KAFKA-17625. When I tried making the `metadata_quorum` default value to use KRaft I ran into some issues getting the tests to work properly, so I created some separate JIRA issues for those tests (The Kafka Streams work is already being tracked here: https://issues.apache.org/jira/browse/KAFKA-17609): - https://issues.apache.org/jira/browse/KAFKA-17915 - https://issues.apache.org/jira/browse/KAFKA-17916 - https://issues.apache.org/jira/browse/KAFKA-17917 My PR also removes `zk` from `quorum.all_non_upgrade` and `quorum.all` so those ZK test variations would be removed, but I don't see that in this PR's diff. -- 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]
