He-Pin opened a new pull request, #980:
URL: https://github.com/apache/pekko-http/pull/980

   ## Motivation
   
   The HTTP/2 persistent-client backoff reconnect test used 
`expectNoMessage(backoffDuration / 2)` to verify that no reconnect occurs 
before the backoff expires. With short configured backoff durations, dividing 
by 2 produced a window too narrow to be reliable on slower CI machines or under 
GC pressure, causing spurious test failures. Upstream akka-http fix: commit 
`b3382778f`.
   
   ## Modification
   
   Change the wait duration from `backoffDuration / 2` to `backoffDuration * 4` 
in `Http2PersistentClientSpec`. The longer window gives the test enough time to 
confirm the backoff is actually being honoured before the connection attempt 
arrives.
   
   Ported from akka-http commit: `b3382778f`
   
   ## Result
   
   The backoff reconnect test no longer produces false-negative (flaky) 
failures caused by a timing window that was too short for slow machines.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to