#19057
89%

Ran 23 Feb 2024 07:53PM UTC

Jobs 1

Files 553

Run time 47s

Badge

Embed ▾

Committed 23 Feb 2024 07:44PM UTC coverage: 88.279% (-0.01%) from 88.291%

Build # #19057

Build Type

push

github

Committed by

ejona86

Commit Message

xds: Avoid nonexistent DNS resolution in XdsClientImplV3Test

The DNS lookups are taking considerable time on the Windows CI (~11s),
which causes the test to time out:

```
Wanted but not invoked:
ldsResourceWatcher.onError(<any>);
-> at io.grpc.xds.XdsClientImplTestBase.sendToNonexistentHost(XdsClientImplTestBase.java:3733)
Actually, there were zero interactions with this mock.

	at io.grpc.xds.XdsClientImplTestBase.sendToNonexistentHost(XdsClientImplTestBase.java:3733)
```

The ARM build, which uses an emulator, has had this test succeed, so the
failure seems unrelated to CPU usage. We want to avoid external I/O
anyway during tests, so removing the DNS lookup is good.

The TSAN comment referenced XdsClientImplTestBase.sendToNonexistentHost,
but the test no longer calls fakeClock.forwardTime so the comment was
out-of-date. Change the comment to make clear the race involved.

Run Details

31129 of 35262 relevant lines covered (88.28%)

0.88 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	#19057.1	23 Feb 2024 07:53PM UTC	0	88.28

Source Files on build #19057

Detailed source file information is not available for this build.

grpc / grpc-java / #19057
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build #19057

grpc / grpc-java / #19057 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build #19057

grpc / grpc-java / #19057
89%

README BADGES
x