search

apple / servicetalk

A networking framework that evolves with your application
https://docs.servicetalk.io
Apache License 2.0
927 stars 181 forks source link

traffic-resilience-http: extend connect timeout for CI even further #3081

Closed idelpivnitskiy closed 2 weeks ago

idelpivnitskiy commented 1 month ago

Motivation:

Even 4 seconds added in #3077 seems too low for CI. Trying higher value. Also, we can reuse already existing CI constant.

bryce-anderson commented 1 month ago

I don't mind extending it further, but it looks like that also didn't work so it might not be a timeout thing.

idelpivnitskiy commented 1 month ago

Looks like increasing the timeout made TrafficResilienceHttpServiceFilterTest even more likely to fails

idelpivnitskiy commented 2 weeks ago

I will close this radar, opened #3093 for minor corrections