[KAFKA-8118] Ensure that tests close ZooKeeper clients since they can impact subsequent tests

• Type: Test
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.3.0, 2.2.1
• Component/s: core
• Assignee: Rajini Sivaram
• Reporter: Rajini Sivaram
• Reviewer: Ismael Juma
• Created: 17/Mar/19 15:46
• Updated: 18/Mar/19 08:53
• Resolved: 18/Mar/19 08:52

Description

A while ago, we added a check to verify that tests are closing ZooKeeper clients in ZooKeeperTestHarness since left over clients can impact subsequent tests (e.g by overwriting static JAAS configuration). But we have changed the ZK client since then and hence the thread name being verified is no longer correct. As a result, we have tests that are leaving ZK clients behind and we need to fix that (as well as the incorrect verification).

Issue Links

• links to: GitHub Pull Request #6456