rreddy-22 commented on code in PR #14582:
URL: https://github.com/apache/kafka/pull/14582#discussion_r1396161519
##########
tests/kafkatest/tests/core/security_rolling_upgrade_test.py:
##########
@@ -126,10 +126,10 @@ def remove_separate_broker_listener(self,
client_security_protocol):
self.bounce()
@cluster(num_nodes=8)
- @matrix(client_protocol=[SecurityConfig.SSL])
+ @matrix(client_protocol=[SecurityConfig.SSL], use_new_coordinator=[True,
False])
Review Comment:
oh yes I must've missed to check in this test, yes zk is the default and
since kraft wasn't passed as the metadata quorum ig all of the tests use zk and
mentioning the group coordinator type is redundant, I'll remove it, thanks for
the catch!
--
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
