tests(*) fix flaky test for locality aware loadbalancing #2564
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Locality aware loadbalancing tests were flaky because they were assuming that config is propagated from CP to DP in 0ms.
I've rewritten and moved the test to TrafficRoutes where we have proper test infra to test loadbalancing.
When I did that, I noticed that we have a problem with fair loadbalancing, for which I filed an issue on Github.
The solution is not in the scope of this PR, with this PR I wanted to fix flaky test.
Issues resolved
No issues
Documentation
Testing
Backwards compatibility
backport-to-stable
label if the code is backwards compatible. Otherwise, list breaking changes.