Skip to content

xds: Fix WRR test flake with TSAN#12523

Merged
ejona86 merged 1 commit intogrpc:masterfrom
ejona86:tsan-wrr
Dec 15, 2025
Merged

xds: Fix WRR test flake with TSAN#12523
ejona86 merged 1 commit intogrpc:masterfrom
ejona86:tsan-wrr

Conversation

@ejona86
Copy link
Copy Markdown
Member

@ejona86 ejona86 commented Nov 19, 2025

Fixes:

org.mockito.exceptions.verification.TooManyActualInvocations:
metricSink.addLongCounter(
    <custom argument matcher>,
    1L,
    [directaddress:///wrr-metrics],
    [, ]
);
Wanted 1 time:
-> at io.grpc.xds.WeightedRoundRobinLoadBalancerTest.metricWithRealChannel(WeightedRoundRobinLoadBalancerTest.java:1307)
But was 2 times:
-> at io.grpc.internal.MetricRecorderImpl.addLongCounter(MetricRecorderImpl.java:103)
-> at io.grpc.internal.MetricRecorderImpl.addLongCounter(MetricRecorderImpl.java:103)

	at io.grpc.xds.WeightedRoundRobinLoadBalancerTest.metricWithRealChannel(WeightedRoundRobinLoadBalancerTest.java:1307)

Fixes:
```
org.mockito.exceptions.verification.TooManyActualInvocations:
metricSink.addLongCounter(
    <custom argument matcher>,
    1L,
    [directaddress:///wrr-metrics],
    [, ]
);
Wanted 1 time:
-> at io.grpc.xds.WeightedRoundRobinLoadBalancerTest.metricWithRealChannel(WeightedRoundRobinLoadBalancerTest.java:1307)
But was 2 times:
-> at io.grpc.internal.MetricRecorderImpl.addLongCounter(MetricRecorderImpl.java:103)
-> at io.grpc.internal.MetricRecorderImpl.addLongCounter(MetricRecorderImpl.java:103)

	at io.grpc.xds.WeightedRoundRobinLoadBalancerTest.metricWithRealChannel(WeightedRoundRobinLoadBalancerTest.java:1307)
```
@ejona86 ejona86 requested a review from AgraVator November 19, 2025 18:20
@ejona86
Copy link
Copy Markdown
Member Author

ejona86 commented Dec 3, 2025

@AgraVator, ping

@ejona86 ejona86 merged commit 24bd540 into grpc:master Dec 15, 2025
15 of 17 checks passed
@ejona86 ejona86 deleted the tsan-wrr branch December 15, 2025 17:41
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants