#20104
89%

Ran 04 Dec 2025 03:38PM UTC

Jobs 1

Files 623

Run time 1min

Badge

Embed ▾

Committed 04 Dec 2025 03:25PM UTC coverage: 88.647% (-0.005%) from 88.652%

Build # #20104

Build Type

push

github

Committed by

ejona86

Commit Message

rls: Avoid missed config update from reentrancy

Since ChildPolicyWrapper() called into the child before
childPolicyMap.put(), it is possible for that child to call back into
RLS and further update state without that child being known. When CDS
is_dynamic=true (since ca99a8c47), it registers the cluster with
XdsDependencyManager, which adds a watch to XdsClient. If XdsClient
already has the results cached then the watch callback can be enqueued
immediately onto the syncContext and execute still within the
constructor.

Calling into the child with the lock held isn't great, as it allows for
this type of reentrancy bug. But that'll take larger changes to fix.

b/464116731

Run Details

35162 of 39665 relevant lines covered (88.65%)

0.89 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
1	88.14	-0.4%	../core/src/main/java/io/grpc/internal/DelayedClientCall.java
1	74.23	-1.03%	../servlet/src/main/java/io/grpc/servlet/AsyncServletOutputStreamWriter.java
1	96.55	-1.72%	../xds/src/main/java/io/grpc/xds/internal/security/certprovider/CertificateProvider.java
5	75.0	-4.31%	../servlet/src/main/java/io/grpc/servlet/ServletServerStream.java
6	79.35	0.23%	../rls/src/main/java/io/grpc/rls/LbPolicyConfiguration.java

Jobs

ID	Job ID	Ran	Files	Coverage
1	#20104.1	04 Dec 2025 03:38PM UTC	623	88.65

grpc / grpc-java / #20104
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build #20104

grpc / grpc-java / #20104 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build #20104

grpc / grpc-java / #20104
89%

README BADGES
x