grpc
/
grpc-java
mirror of https://github.com/grpc/grpc-java.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

xds: ignore zero-weighted locality

This commit is contained in:
ZHANG Dapeng 2019-07-08 15:21:52 -07:00 committed by GitHub
parent 722c66ef91
commit eecd5a72bd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 17 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
xds/src/main/java/io/grpc/xds/XdsComms.java
View File

@ -251,8 +251,10 @@ final class XdsComms {
}
int localityWeight = localityLbEndpoints.getLoadBalancingWeight().getValue();
localityEndpointsMapping.put(
locality, new LocalityInfo(lbEndPoints, localityWeight));
if (localityWeight != 0) {
localityEndpointsMapping.put(
locality, new LocalityInfo(lbEndPoints, localityWeight));
}
}
localityEndpointsMapping = Collections.unmodifiableMap(localityEndpointsMapping);

13
xds/src/test/java/io/grpc/xds/XdsCommsTest.java
View File

@ -256,6 +256,15 @@ public class XdsCommsTest {
.setAddress("addr22").setPortValue(22))))
.setLoadBalancingWeight(UInt32Value.of(22))
.build();
Locality localityProto3 = Locality.newBuilder()
.setRegion("region3").setZone("zone3").setSubZone("subzone3").build();
LbEndpoint endpoint3 = LbEndpoint.newBuilder()
.setEndpoint(Endpoint.newBuilder()
.setAddress(Address.newBuilder()
.setSocketAddress(SocketAddress.newBuilder()
.setAddress("addr31").setPortValue(31))))
.setLoadBalancingWeight(UInt32Value.of(31))
.build();
DiscoveryResponse edsResponse = DiscoveryResponse.newBuilder()
.addResources(Any.pack(ClusterLoadAssignment.newBuilder()
.addEndpoints(LocalityLbEndpoints.newBuilder()
@ -268,6 +277,10 @@ public class XdsCommsTest {
.addLbEndpoints(endpoint21)
.addLbEndpoints(endpoint22)
.setLoadBalancingWeight(UInt32Value.of(2)))
.addEndpoints(LocalityLbEndpoints.newBuilder()
.setLocality(localityProto3)
.addLbEndpoints(endpoint3)
.setLoadBalancingWeight(UInt32Value.of(0)))
.build()))
.setTypeUrl(EDS_TYPE_URL)
.build();