From 8bb6a5fa9c280df784825e270068be85569265f6 Mon Sep 17 00:00:00 2001 From: Hong Zhiguo Date: Thu, 12 Jul 2018 11:29:23 +0800 Subject: [PATCH] update TestOnlyLocalNodePorts to make sure only add local RS --- pkg/proxy/ipvs/proxier_test.go | 34 ++++++++++++++++++++++++---------- 1 file changed, 24 insertions(+), 10 deletions(-) diff --git a/pkg/proxy/ipvs/proxier_test.go b/pkg/proxy/ipvs/proxier_test.go index 659bc98932a..6041668ab36 100644 --- a/pkg/proxy/ipvs/proxier_test.go +++ b/pkg/proxy/ipvs/proxier_test.go @@ -1001,18 +1001,32 @@ func TestOnlyLocalNodePorts(t *testing.T) { ) epIP := "10.180.0.1" + epIP1 := "10.180.1.1" + thisHostname := testHostname + otherHostname := "other-hostname" + makeEndpointsMap(fp, makeTestEndpoints(svcPortName.Namespace, svcPortName.Name, func(ept *api.Endpoints) { - ept.Subsets = []api.EndpointSubset{{ - Addresses: []api.EndpointAddress{{ - IP: epIP, - NodeName: nil, - }}, - Ports: []api.EndpointPort{{ - Name: svcPortName.Port, - Port: int32(svcPort), - }}, - }} + ept.Subsets = []api.EndpointSubset{ + { // **local** endpoint address, should be added as RS + Addresses: []api.EndpointAddress{{ + IP: epIP, + NodeName: &thisHostname, + }}, + Ports: []api.EndpointPort{{ + Name: svcPortName.Port, + Port: int32(svcPort), + }}}, + { // **remote** endpoint address, should not be added as RS + Addresses: []api.EndpointAddress{{ + IP: epIP1, + NodeName: &otherHostname, + }}, + Ports: []api.EndpointPort{{ + Name: svcPortName.Port, + Port: int32(svcPort), + }}, + }} }), )