KAFKA-13129: replace describe topic via zk with describe users #11115
Conversation
|
@ijuma @rondagostino , please help take a look. Thank you. Verified in my local env that the test verification can be passed. |
| @@ -220,16 +220,15 @@ def create(self, path, chroot=None, value=""): | |||
| output = self.nodes[0].account.ssh_output(cmd) | |||
| self.logger.debug(output) | |||
|
|
|||
| def describe(self, topic): | |||
| def describeUsers(self): | |||
There was a problem hiding this comment.
replace the original describe topic method with describeUsers since describe topic via zk is not supported anymore.
|
@showuon |
|
@ijuma , I have to admit that I cannot complete the I checked the system tests results that run by @rondagostino in this PR: #10982 before the Sorry, and thank you. |
Replace the unsupported describe topic via zk with describe users to fix the system tests. For the upgrade_test case where TLS support is not required, use list_acls instead. Reviewers: Ismael Juma <[email protected]>
|
No worries, I ran a few variations and they look good. Also cherry-picked this to 3.0. |
…e#11115) Replace the unsupported describe topic via zk with describe users to fix the system tests. For the upgrade_test case where TLS support is not required, use list_acls instead. Reviewers: Ismael Juma <[email protected]>
Replace the unsupported describe topic via zk with describe users to fix the system tests.
For the
upgrade_testcase where TLS support is not required, uselist_aclsinstead.Committer Checklist (excluded from commit message)