Mickael Maison

Shouldn't we instead update the tests to use temp directories and cleanup any files they create?

Thanks for the PR. The [test failures](https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-15175/1/testReport/) look related as `SaslSslAdminIntegrationTest` and `SslAdminIntegrationTest` both extend `BaseAdminIntegrationTest`

Hi @CalvinConfluent, Thanks for the clarification. The issue is that this causes a compatibility change. For example with the following code: ```java DescribeTopicsOptions options = new DescribeTopicsOptions().includeAuthorizedOperations(false); TopicCollection topics =...

I created https://issues.apache.org/jira/browse/KAFKA-16865

The CI failed due to timeout. I kicked another build.

None of the test failures are related, merging to trunk

Having a method to clear the registry would be really helpful for tests. Sure we can inject a new `PrometheusRegistry` instance but that requires further refactoring to adapt code that...

cc @C0urante as you also voted on the KIP

> I suggest to proceed and merge java version of command in core module. This can be helpful while re-implementing these classes but I'm not sure we want to merge...

I agree that the current behavior of `plugin.path` does not quite match the documentation. I confirmed that plugins can be loaded regardless of the number of nested directories they are...