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-16053: Fix leaks of KDC server in tests #15079
Conversation
@dajac @jolshan please review. This is not the root cause of our recent OOM errors but is a contributor to it. (cc: @stanislavkozlovski as a potential backport to 3.7) |
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.
There are a lot of existing failures on this run -- but I didn't see any for the tests modified here.
This commit closes the KDC server properly in `CustomQuotaCallbackTest` and `AclAuthorizerWithZkSaslTest`. Reviewers: Justine Olshan <jolshan@confluent.io>
This commit closes the KDC server properly in `CustomQuotaCallbackTest` and `AclAuthorizerWithZkSaslTest`. Reviewers: Justine Olshan <jolshan@confluent.io>
Nice catch! |
backported to 3.6 and 3.7 branch |
This commit closes the KDC server properly in `CustomQuotaCallbackTest` and `AclAuthorizerWithZkSaslTest`. Reviewers: Justine Olshan <jolshan@confluent.io>
This commit closes the KDC server properly in `CustomQuotaCallbackTest` and `AclAuthorizerWithZkSaslTest`. Reviewers: Justine Olshan <jolshan@confluent.io>
… (#497) This commit closes the KDC server properly in `CustomQuotaCallbackTest` and `AclAuthorizerWithZkSaslTest`. Reviewers: Justine Olshan <jolshan@confluent.io> Co-authored-by: Divij Vaidya <diviv@amazon.com>
This commit closes the KDC server properly in `CustomQuotaCallbackTest` and `AclAuthorizerWithZkSaslTest`. Reviewers: Justine Olshan <jolshan@confluent.io>
This commit closes the KDC server properly in `CustomQuotaCallbackTest` and `AclAuthorizerWithZkSaslTest`. Reviewers: Justine Olshan <jolshan@confluent.io>
Problem
We are facing OOM while running test suite for Apache Kafka as discussed in https://lists.apache.org/thread/d5js0xpsrsvhgjb10mbzo9cwsy8087x4
Changes
This JIRA fixes ~123MB of heap leak caused by DefaultDirectoryService objects in the heap memory. The
DefaultDirectoryService
is used byMiniKdc.scala
. This change fixes a couple of leaks where the kdc service is not closed at the end of test run.