• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

grpc / grpc-java / #19191
89%

Build:
DEFAULT BRANCH: master
Ran 05 May 2024 08:04PM UTC
Jobs 1
Files 571
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

03 May 2024 07:42PM UTC coverage: 88.323% (+0.02%) from 88.303%
#19191

push

github

ejona86
xds: Plumb locality in xds_cluster_impl and weighted_target

As part of gRFC A78:

> To support the locality label in the WRR metrics, we will extend the
> `weighted_target` LB policy (see A28) to define a resolver attribute
> that indicates the name of its child. This attribute will be passed
> down to each of its children with the appropriate value, so that any
> LB policy that sits underneath the `weighted_target` policy will be
> able to use it.

xds_cluster_impl is involved because it uses the child names in the
AddressFilter, which must match the names used by weighted_target.
Instead of using Locality.toString() in multiple policies and assuming
the policies agree, we now have xds_cluster_impl decide the locality's
name and pass it down explicitly. This allows us to change the name
format to match gRFC A78:

> If locality information is available, the value of this label will be
> of the form `{region="${REGION}", zone="${ZONE}",
> sub_zone="${SUB_ZONE}"}`, where `${REGION}`, `${ZONE}`, and
> `${SUB_ZONE}` are replaced with the actual values. If no locality
> information is available, the label will be set to the empty string.

31520 of 35687 relevant lines covered (88.32%)

0.88 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
2
83.16
0.0% ../okhttp/src/main/java/io/grpc/okhttp/ExceptionHandlingFrameWriter.java
4
96.26
0.18% ../xds/src/main/java/io/grpc/xds/WeightedTargetLoadBalancer.java
4
88.52
0.0% ../xds/src/main/java/io/grpc/xds/WrrLocalityLoadBalancer.java
9
95.63
0.05% ../xds/src/main/java/io/grpc/xds/ClusterResolverLoadBalancer.java
10
93.75
-0.24% ../xds/src/main/java/io/grpc/xds/ClusterImplLoadBalancer.java
Jobs
ID Job ID Ran Files Coverage
1 #19191.1 05 May 2024 08:04PM UTC 571
88.32
Source Files on build #19191
  • Tree
  • List 571
  • Changed 9
  • Source Changed 0
  • Coverage Changed 9
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #19191
  • 077dcbf9 on github
  • Prev Build on master
  • Next Build on master
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc