envoy icon indicating copy to clipboard operation
envoy copied to clipboard

`RateLimitQuotaIntegrationTest.TwoRequestWithTokenBucketThrottling` is flakey (TIMEOUT)

Open phlax opened this issue 1 year ago • 2 comments

This test was added in https://github.com/envoyproxy/envoy/pull/33921

as that PR was addressing a different flake im reluctant to revert, but the test takes a very long time and ci analytics gives it a 0% pass rate

image

https://dev.azure.com/cncf/envoy/_build/results?buildId=170010&view=logs&j=767be981-567e-57d8-68c3-2140ede0a0bd&t=2181edf2-f610-59f2-c43a-04bb9d0bca00&l=457

phlax avatar May 09 '24 09:05 phlax

The newly added test is flaky actually due to the same reason: pending stats implementation

I am thinking that remove the newly added test and lower the coverage threshold for RLQS

tyxia avatar May 09 '24 09:05 tyxia

thanks @tyxia - probably we should do that as apart from failing its taking a long time

perhaps we should add a ticket re RLQS coverage?

phlax avatar May 09 '24 09:05 phlax