-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
KAFKA-18677: Update ConsoleConsumerTest system test #18763
Conversation
65e7451
to
357f0c1
Compare
@@ -44,8 +44,7 @@ def setUp(self): | |||
@cluster(num_nodes=3) | |||
@matrix(security_protocol=['PLAINTEXT', 'SSL'], metadata_quorum=quorum.all_kraft) | |||
@cluster(num_nodes=4) | |||
@matrix(security_protocol=['SASL_SSL'], sasl_mechanism=['PLAIN'], metadata_quorum=quorum.all_kraft) | |||
@matrix(security_protocol=['SASL_SSL'], sasl_mechanism=['SCRAM-SHA-256', 'SCRAM-SHA-512']) # SCRAM not yet supported with KRaft | |||
@matrix(security_protocol=['SASL_SSL'], sasl_mechanism=['PLAIN', 'SCRAM-SHA-256', 'SCRAM-SHA-512'], metadata_quorum=quorum.all_kraft) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While we are here, I wonder if we could directly remove the usage of ZookeeperService
in this class. What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I'll give it a try.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JIRA: KAFKA-18677 The ConsoleConsumerTest system test still relied on ZK so it fails. We should convert it to kraft.
357f0c1
to
15e5a52
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This patch converts the ConsoleConsumerTest system test to only use KRaft. Reviewers: David Jacot <[email protected]>
Merged to trunk and 4.0. Thanks. |
This patch converts the ConsoleConsumerTest system test to only use KRaft. Reviewers: David Jacot <[email protected]>
This patch converts the ConsoleConsumerTest system test to only use KRaft. Reviewers: David Jacot <[email protected]>
JIRA: KAFKA-18677
The ConsoleConsumerTest system test still relied on ZK, so it failed.
We should convert it to kraft.
Committer Checklist (excluded from commit message)