Re: [PR] KAFKA-14588 Tests for ConfigCommand of DynamicBrokerReconfigurationTest rewritten in java [kafka]

2024-08-25 Thread via GitHub
github-actions[bot] commented on PR #15848: URL: https://github.com/apache/kafka/pull/15848#issuecomment-2309238540 This PR is being marked as stale since it has not had any activity in 90 days. If you would like to keep this PR alive, please ask a committer for review. If the PR has merge

Re: [PR] KAFKA-14588 Tests for ConfigCommand of DynamicBrokerReconfigurationTest rewritten in java [kafka]

2024-05-27 Thread via GitHub
nizhikov commented on PR #15848: URL: https://github.com/apache/kafka/pull/15848#issuecomment-2133050783 Got it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubsc

Re: [PR] KAFKA-14588 Tests for ConfigCommand of DynamicBrokerReconfigurationTest rewritten in java [kafka]

2024-05-27 Thread via GitHub
chia7712 commented on PR #15848: URL: https://github.com/apache/kafka/pull/15848#issuecomment-2133045240 I just notice that this test requires `SaslSetup`. the new test infra does not support that for now (https://issues.apache.org/jira/browse/KAFKA-16680). Hence, I'd like to pending this r

Re: [PR] KAFKA-14588 Tests for ConfigCommand of DynamicBrokerReconfigurationTest rewritten in java [kafka]

2024-05-24 Thread via GitHub
nizhikov commented on PR #15848: URL: https://github.com/apache/kafka/pull/15848#issuecomment-2130911519 CI looks OK for me. `DynamicBrokerReconfigurationTest#testTrustStoreAlter` fails in trunk, also. https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka/detail

Re: [PR] KAFKA-14588 Tests for ConfigCommand of DynamicBrokerReconfigurationTest rewritten in java [kafka]

2024-05-21 Thread via GitHub
nizhikov commented on PR #15848: URL: https://github.com/apache/kafka/pull/15848#issuecomment-2122859856 @chia7712 PR ready for review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

[PR] KAFKA-14588 Tests for ConfigCommand of DynamicBrokerReconfigurationTest rewritten in java [kafka]

2024-05-02 Thread via GitHub
nizhikov opened a new pull request, #15848: URL: https://github.com/apache/kafka/pull/15848 This is fourth part of https://github.com/apache/kafka/pull/15417 refactoring. PR intention - split big PR in parts to simplify review. PR contains tests of `DynamicBrokerReconfigurationTest` th