lh0156 opened a new pull request, #22943:
URL: https://github.com/apache/kafka/pull/22943

   ## Summary
   
   - Wait for the channel to be fully ready before testing idle connection 
expiry.
   - Remove the TLS 1.3-specific flaky-test override so the shared test runs 
normally for TLS 1.3.
   
   ## Motivation
   
   Fixes [KAFKA-14249](https://issues.apache.org/jira/browse/KAFKA-14249).
   
   The idle-expiry test only waited for the TCP connection to reach 
`CONNECTED`. With TLS 1.3, the first subsequent poll can still process 
handshake I/O and refresh the selector's last-active timestamp. The test then 
misses the expiry deadline and was marked flaky and excluded from normal test 
execution.
   
   Waiting for `READY` makes the test precondition match the behavior being 
tested: no handshake work remains before the idle period starts. This lets the 
TLS 1.3 variant run as a regular regression test.
   
   ## Validation
   
   - Ran the TLS 1.3 selector suite with `testCloseOldestConnection` included; 
it passed.
   - Ran `SelectorTest`, `Tls12SelectorTest`, and `Tls13SelectorTest`; all 
passed.
   - Ran `./gradlew :clients:test --no-build-cache --console=plain` 
successfully.
   - Ran `git diff --check` successfully.
   


-- 
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]

Reply via email to