Skip to content

Commit f621bd8

Browse files
mihaitodorrelistan
authored andcommitted
Rename loadAssignment -> assignment
1 parent 17251cc commit f621bd8

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

envoy/adapter/adapter.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -93,9 +93,9 @@ func EnvoyResourcesFromState(state *catalog.ServicesState, bindIP string,
9393

9494
envoyServiceName := SvcName(svc.Name, port.ServicePort)
9595

96-
if loadAssignment, ok := endpointMap[envoyServiceName]; ok {
97-
loadAssignment.Endpoints[0].LbEndpoints =
98-
append(loadAssignment.Endpoints[0].LbEndpoints,
96+
if assignment, ok := endpointMap[envoyServiceName]; ok {
97+
assignment.Endpoints[0].LbEndpoints =
98+
append(assignment.Endpoints[0].LbEndpoints,
9999
envoyServiceFromService(svc, port.ServicePort, useHostnames)...)
100100
} else {
101101
endpointMap[envoyServiceName] = &api.ClusterLoadAssignment{

envoy/server_test.go

+9-9
Original file line numberDiff line numberDiff line change
@@ -77,13 +77,13 @@ func validateListener(serialisedListener *any.Any, svc service.Service) {
7777
}
7878
}
7979

80-
func validateEndpoints(serialisedLoadAssignment *any.Any, svc service.Service) {
81-
loadAssignment := &api.ClusterLoadAssignment{}
82-
err := ptypes.UnmarshalAny(serialisedLoadAssignment, loadAssignment)
80+
func validateEndpoints(serialisedAssignment *any.Any, svc service.Service) {
81+
assignment := &api.ClusterLoadAssignment{}
82+
err := ptypes.UnmarshalAny(serialisedAssignment, assignment)
8383
So(err, ShouldBeNil)
84-
So(loadAssignment.GetClusterName(), ShouldEqual, adapter.SvcName(svc.Name, svc.Ports[0].ServicePort))
84+
So(assignment.GetClusterName(), ShouldEqual, adapter.SvcName(svc.Name, svc.Ports[0].ServicePort))
8585

86-
localityEndpoints := loadAssignment.GetEndpoints()
86+
localityEndpoints := assignment.GetEndpoints()
8787
So(localityEndpoints, ShouldHaveLength, 1)
8888

8989
endpoints := localityEndpoints[0].GetLbEndpoints()
@@ -296,12 +296,12 @@ func Test_PortForServicePort(t *testing.T) {
296296

297297
resources := envoyMock.GetResource(stream, cache.EndpointType, state.Hostname)
298298
So(resources, ShouldHaveLength, 1)
299-
loadAssignment := &api.ClusterLoadAssignment{}
300-
err := ptypes.UnmarshalAny(resources[0], loadAssignment)
299+
assignment := &api.ClusterLoadAssignment{}
300+
err := ptypes.UnmarshalAny(resources[0], assignment)
301301
So(err, ShouldBeNil)
302-
So(loadAssignment.GetEndpoints(), ShouldHaveLength, 1)
302+
So(assignment.GetEndpoints(), ShouldHaveLength, 1)
303303
var ports sort.IntSlice
304-
for _, endpoint := range loadAssignment.GetEndpoints()[0].GetLbEndpoints() {
304+
for _, endpoint := range assignment.GetEndpoints()[0].GetLbEndpoints() {
305305
ports = append(ports,
306306
int(endpoint.GetEndpoint().GetAddress().GetSocketAddress().GetPortValue()))
307307
}

0 commit comments

Comments
 (0)