chickenchickenlove opened a new pull request, #21280: URL: https://github.com/apache/kafka/pull/21280
This PR fixes a flaky failure in `ConfigCommandIntegrationTest.testUpdateInvalidBrokerConfigs`. The test updates an unknown broker dynamic config `(invalid=2)` and immediately runs `kafka-configs --describe` to assert that the unknown config is treated as sensitive `(sensitive=true)` and returned as `invalid=null`. In KRaft, the alter request can return before the change is fully visible to `--describe` because `dynamic config` updates are applied asynchronously and published via `DynamicConfigPublisher` as metadata updates are processed. As a result, the first describe may read stale state and miss the expected fields, causing intermittent failures. To make the test robust, we now poll `--describe` for a short period and proceed once the expected output becomes visible (or fail with the last observed output on timeout). -- 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 unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
