#19835
89%

Ran 30 May 2025 04:10PM UTC

Jobs 1

Files 615

Run time 1min

Badge

Embed ▾

Committed 30 May 2025 04:00PM UTC coverage: 88.599% (-0.005%) from 88.604%

Build # #19835

Build Type

push

github

Committed by

web-flow

Commit Message

xds: Improve shutdown handling of XdsDepManager

The most important change here is to handle subscribeToCluster() calls
after shutdown(), and preventing the internal state from being heavily
confused as the assumption is there are no watchers after shutdown().

ClusterSubscription.closed isn't strictly necessary, but I don't want
the code to depend on double-deregistration being safe.
maybePublishConfig() isn't being called after shutdown(), but adding the
protection avoids a class of bugs that would cause channel panic.

Run Details

34793 of 39270 relevant lines covered (88.6%)

0.89 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
1	91.37	-0.21%	../netty/src/main/java/io/grpc/netty/NettyServerHandler.java
3	95.07	-0.44%	../core/src/main/java/io/grpc/internal/RetriableStream.java
3	92.38	-0.95%	../xds/src/main/java/io/grpc/xds/client/ControlPlaneClient.java
38	91.52	-0.25%	../xds/src/main/java/io/grpc/xds/XdsDependencyManager.java

Jobs

ID	Job ID	Ran	Files	Coverage
1	#19835.1	30 May 2025 04:10PM UTC	615	88.6

grpc / grpc-java / #19835
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build #19835

grpc / grpc-java / #19835 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build #19835

grpc / grpc-java / #19835
89%

README BADGES
x