mirror of
https://github.com/ceph/ceph-csi.git
synced 2024-12-22 21:10:22 +00:00
1292 lines
37 KiB
Go
1292 lines
37 KiB
Go
|
/*
|
||
|
|
Copyright 2017 The Kubernetes Authors.
|
||
|
|
|
||
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
||
|
|
you may not use this file except in compliance with the License.
|
||
|
|
You may obtain a copy of the License at
|
||
|
|
|
||
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
||
|
|
|
||
|
|
Unless required by applicable law or agreed to in writing, software
|
||
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
||
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||
|
|
See the License for the specific language governing permissions and
|
||
|
|
limitations under the License.
|
||
|
|
*/
|
||
|
|
|
||
|
|
package proxy
|
||
|
|
|
||
|
|
import (
|
||
|
|
"reflect"
|
||
|
|
"testing"
|
||
|
|
|
||
|
|
"github.com/davecgh/go-spew/spew"
|
||
|
|
|
||
|
|
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
|
||
|
|
"k8s.io/apimachinery/pkg/types"
|
||
|
|
"k8s.io/apimachinery/pkg/util/sets"
|
||
|
|
api "k8s.io/kubernetes/pkg/apis/core"
|
||
|
|
)
|
||
|
|
|
||
|
|
func (proxier *FakeProxier) addEndpoints(endpoints *api.Endpoints) {
|
||
|
|
proxier.endpointsChanges.Update(nil, endpoints)
|
||
|
|
}
|
||
|
|
|
||
|
|
func (proxier *FakeProxier) updateEndpoints(oldEndpoints, endpoints *api.Endpoints) {
|
||
|
|
proxier.endpointsChanges.Update(oldEndpoints, endpoints)
|
||
|
|
}
|
||
|
|
|
||
|
|
func (proxier *FakeProxier) deleteEndpoints(endpoints *api.Endpoints) {
|
||
|
|
proxier.endpointsChanges.Update(endpoints, nil)
|
||
|
|
}
|
||
|
|
|
||
|
|
func TestGetLocalEndpointIPs(t *testing.T) {
|
||
|
|
testCases := []struct {
|
||
|
|
endpointsMap EndpointsMap
|
||
|
|
expected map[types.NamespacedName]sets.String
|
||
|
|
}{{
|
||
|
|
// Case[0]: nothing
|
||
|
|
endpointsMap: EndpointsMap{},
|
||
|
|
expected: map[types.NamespacedName]sets.String{},
|
||
|
|
}, {
|
||
|
|
// Case[1]: unnamed port
|
||
|
|
endpointsMap: EndpointsMap{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expected: map[types.NamespacedName]sets.String{},
|
||
|
|
}, {
|
||
|
|
// Case[2]: unnamed port local
|
||
|
|
endpointsMap: EndpointsMap{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "1.1.1.1:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expected: map[types.NamespacedName]sets.String{
|
||
|
|
{Namespace: "ns1", Name: "ep1"}: sets.NewString("1.1.1.1"),
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[3]: named local and non-local ports for the same IP.
|
||
|
|
endpointsMap: EndpointsMap{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
&BaseEndpointInfo{Endpoint: "1.1.1.2:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "1.1.1.1:12", IsLocal: false},
|
||
|
|
&BaseEndpointInfo{Endpoint: "1.1.1.2:12", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expected: map[types.NamespacedName]sets.String{
|
||
|
|
{Namespace: "ns1", Name: "ep1"}: sets.NewString("1.1.1.2"),
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[4]: named local and non-local ports for different IPs.
|
||
|
|
endpointsMap: EndpointsMap{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns2", "ep2", "p22"): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "2.2.2.2:22", IsLocal: true},
|
||
|
|
&BaseEndpointInfo{Endpoint: "2.2.2.22:22", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns2", "ep2", "p23"): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "2.2.2.3:23", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns4", "ep4", "p44"): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "4.4.4.4:44", IsLocal: true},
|
||
|
|
&BaseEndpointInfo{Endpoint: "4.4.4.5:44", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns4", "ep4", "p45"): []Endpoint{
|
||
|
|
&BaseEndpointInfo{Endpoint: "4.4.4.6:45", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expected: map[types.NamespacedName]sets.String{
|
||
|
|
{Namespace: "ns2", Name: "ep2"}: sets.NewString("2.2.2.2", "2.2.2.22", "2.2.2.3"),
|
||
|
|
{Namespace: "ns4", Name: "ep4"}: sets.NewString("4.4.4.4", "4.4.4.6"),
|
||
|
|
},
|
||
|
|
}}
|
||
|
|
|
||
|
|
for tci, tc := range testCases {
|
||
|
|
// outputs
|
||
|
|
localIPs := GetLocalEndpointIPs(tc.endpointsMap)
|
||
|
|
|
||
|
|
if !reflect.DeepEqual(localIPs, tc.expected) {
|
||
|
|
t.Errorf("[%d] expected %#v, got %#v", tci, tc.expected, localIPs)
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|
||
|
|
|
||
|
|
func makeTestEndpoints(namespace, name string, eptFunc func(*api.Endpoints)) *api.Endpoints {
|
||
|
|
ept := &api.Endpoints{
|
||
|
|
ObjectMeta: metav1.ObjectMeta{
|
||
|
|
Name: name,
|
||
|
|
Namespace: namespace,
|
||
|
|
},
|
||
|
|
}
|
||
|
|
eptFunc(ept)
|
||
|
|
return ept
|
||
|
|
}
|
||
|
|
|
||
|
|
// This is a coarse test, but it offers some modicum of confidence as the code is evolved.
|
||
|
|
func TestEndpointsToEndpointsMap(t *testing.T) {
|
||
|
|
epTracker := NewEndpointChangeTracker("test-hostname", nil, nil, nil)
|
||
|
|
|
||
|
|
trueVal := true
|
||
|
|
falseVal := false
|
||
|
|
|
||
|
|
testCases := []struct {
|
||
|
|
desc string
|
||
|
|
newEndpoints *api.Endpoints
|
||
|
|
expected map[ServicePortName][]*BaseEndpointInfo
|
||
|
|
isIPv6Mode *bool
|
||
|
|
}{
|
||
|
|
{
|
||
|
|
desc: "nothing",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "no changes, unnamed port",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "no changes, named port",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "port",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "port"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "new port",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "remove port",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "new IP and port",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}, {
|
||
|
|
IP: "2.2.2.2",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p1",
|
||
|
|
Port: 11,
|
||
|
|
}, {
|
||
|
|
Name: "p2",
|
||
|
|
Port: 22,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p1"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
{Endpoint: "2.2.2.2:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p2"): {
|
||
|
|
{Endpoint: "1.1.1.1:22", IsLocal: false},
|
||
|
|
{Endpoint: "2.2.2.2:22", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "remove IP and port",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p1",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p1"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "rename port",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p2",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p2"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "renumber port",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p1",
|
||
|
|
Port: 22,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p1"): {
|
||
|
|
{Endpoint: "1.1.1.1:22", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "should omit IPv6 address in IPv4 mode",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}, {
|
||
|
|
IP: "2001:db8:85a3:0:0:8a2e:370:7334",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p1",
|
||
|
|
Port: 11,
|
||
|
|
}, {
|
||
|
|
Name: "p2",
|
||
|
|
Port: 22,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p1"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p2"): {
|
||
|
|
{Endpoint: "1.1.1.1:22", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
isIPv6Mode: &falseVal,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
desc: "should omit IPv4 address in IPv6 mode",
|
||
|
|
newEndpoints: makeTestEndpoints("ns1", "ep1", func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{
|
||
|
|
{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}, {
|
||
|
|
IP: "2001:db8:85a3:0:0:8a2e:370:7334",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p1",
|
||
|
|
Port: 11,
|
||
|
|
}, {
|
||
|
|
Name: "p2",
|
||
|
|
Port: 22,
|
||
|
|
}},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
}),
|
||
|
|
expected: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p1"): {
|
||
|
|
{Endpoint: "[2001:db8:85a3:0:0:8a2e:370:7334]:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p2"): {
|
||
|
|
{Endpoint: "[2001:db8:85a3:0:0:8a2e:370:7334]:22", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
isIPv6Mode: &trueVal,
|
||
|
|
},
|
||
|
|
}
|
||
|
|
|
||
|
|
for _, tc := range testCases {
|
||
|
|
epTracker.isIPv6Mode = tc.isIPv6Mode
|
||
|
|
// outputs
|
||
|
|
newEndpoints := epTracker.endpointsToEndpointsMap(tc.newEndpoints)
|
||
|
|
|
||
|
|
if len(newEndpoints) != len(tc.expected) {
|
||
|
|
t.Errorf("[%s] expected %d new, got %d: %v", tc.desc, len(tc.expected), len(newEndpoints), spew.Sdump(newEndpoints))
|
||
|
|
}
|
||
|
|
for x := range tc.expected {
|
||
|
|
if len(newEndpoints[x]) != len(tc.expected[x]) {
|
||
|
|
t.Errorf("[%s] expected %d endpoints for %v, got %d", tc.desc, len(tc.expected[x]), x, len(newEndpoints[x]))
|
||
|
|
} else {
|
||
|
|
for i := range newEndpoints[x] {
|
||
|
|
ep := newEndpoints[x][i].(*BaseEndpointInfo)
|
||
|
|
if *ep != *(tc.expected[x][i]) {
|
||
|
|
t.Errorf("[%s] expected new[%v][%d] to be %v, got %v", tc.desc, x, i, tc.expected[x][i], *ep)
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|
||
|
|
|
||
|
|
func TestUpdateEndpointsMap(t *testing.T) {
|
||
|
|
var nodeName = testHostname
|
||
|
|
|
||
|
|
emptyEndpoint := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{}
|
||
|
|
}
|
||
|
|
unnamedPort := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
unnamedPortLocal := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
namedPortLocal := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
namedPort := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
namedPortRenamed := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11-2",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
namedPortRenumbered := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 22,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
namedPortsLocalNoLocal := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}, {
|
||
|
|
IP: "1.1.1.2",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}, {
|
||
|
|
Name: "p12",
|
||
|
|
Port: 12,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
multipleSubsets := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}, {
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.2",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p12",
|
||
|
|
Port: 12,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
multipleSubsetsWithLocal := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}, {
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.2",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p12",
|
||
|
|
Port: 12,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
multipleSubsetsMultiplePortsLocal := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}, {
|
||
|
|
Name: "p12",
|
||
|
|
Port: 12,
|
||
|
|
}},
|
||
|
|
}, {
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.3",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p13",
|
||
|
|
Port: 13,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
multipleSubsetsIPsPorts1 := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}, {
|
||
|
|
IP: "1.1.1.2",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}, {
|
||
|
|
Name: "p12",
|
||
|
|
Port: 12,
|
||
|
|
}},
|
||
|
|
}, {
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.3",
|
||
|
|
}, {
|
||
|
|
IP: "1.1.1.4",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p13",
|
||
|
|
Port: 13,
|
||
|
|
}, {
|
||
|
|
Name: "p14",
|
||
|
|
Port: 14,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
multipleSubsetsIPsPorts2 := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "2.2.2.1",
|
||
|
|
}, {
|
||
|
|
IP: "2.2.2.2",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p21",
|
||
|
|
Port: 21,
|
||
|
|
}, {
|
||
|
|
Name: "p22",
|
||
|
|
Port: 22,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
complexBefore1 := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
complexBefore2 := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "2.2.2.2",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}, {
|
||
|
|
IP: "2.2.2.22",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p22",
|
||
|
|
Port: 22,
|
||
|
|
}},
|
||
|
|
}, {
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "2.2.2.3",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p23",
|
||
|
|
Port: 23,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
complexBefore4 := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "4.4.4.4",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}, {
|
||
|
|
IP: "4.4.4.5",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p44",
|
||
|
|
Port: 44,
|
||
|
|
}},
|
||
|
|
}, {
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "4.4.4.6",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p45",
|
||
|
|
Port: 45,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
complexAfter1 := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.1",
|
||
|
|
}, {
|
||
|
|
IP: "1.1.1.11",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p11",
|
||
|
|
Port: 11,
|
||
|
|
}},
|
||
|
|
}, {
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "1.1.1.2",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p12",
|
||
|
|
Port: 12,
|
||
|
|
}, {
|
||
|
|
Name: "p122",
|
||
|
|
Port: 122,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
complexAfter3 := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "3.3.3.3",
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p33",
|
||
|
|
Port: 33,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
complexAfter4 := func(ept *api.Endpoints) {
|
||
|
|
ept.Subsets = []api.EndpointSubset{{
|
||
|
|
Addresses: []api.EndpointAddress{{
|
||
|
|
IP: "4.4.4.4",
|
||
|
|
NodeName: &nodeName,
|
||
|
|
}},
|
||
|
|
Ports: []api.EndpointPort{{
|
||
|
|
Name: "p44",
|
||
|
|
Port: 44,
|
||
|
|
}},
|
||
|
|
}}
|
||
|
|
}
|
||
|
|
|
||
|
|
testCases := []struct {
|
||
|
|
// previousEndpoints and currentEndpoints are used to call appropriate
|
||
|
|
// handlers OnEndpoints* (based on whether corresponding values are nil
|
||
|
|
// or non-nil) and must be of equal length.
|
||
|
|
previousEndpoints []*api.Endpoints
|
||
|
|
currentEndpoints []*api.Endpoints
|
||
|
|
oldEndpoints map[ServicePortName][]*BaseEndpointInfo
|
||
|
|
expectedResult map[ServicePortName][]*BaseEndpointInfo
|
||
|
|
expectedStaleEndpoints []ServiceEndpoint
|
||
|
|
expectedStaleServiceNames map[ServicePortName]bool
|
||
|
|
expectedHealthchecks map[types.NamespacedName]int
|
||
|
|
}{{
|
||
|
|
// Case[0]: nothing
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
}, {
|
||
|
|
// Case[1]: no change, unnamed port
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", unnamedPort),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", unnamedPort),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
}, {
|
||
|
|
// Case[2]: no change, named port, local
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPortLocal),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPortLocal),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{
|
||
|
|
makeNSN("ns1", "ep1"): 1,
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[3]: no change, multiple subsets
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", multipleSubsets),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", multipleSubsets),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
}, {
|
||
|
|
// Case[4]: no change, multiple subsets, multiple ports, local
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", multipleSubsetsMultiplePortsLocal),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", multipleSubsetsMultiplePortsLocal),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.1:12", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p13"): {
|
||
|
|
{Endpoint: "1.1.1.3:13", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.1:12", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p13"): {
|
||
|
|
{Endpoint: "1.1.1.3:13", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{
|
||
|
|
makeNSN("ns1", "ep1"): 1,
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[5]: no change, multiple endpoints, subsets, IPs, and ports
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", multipleSubsetsIPsPorts1),
|
||
|
|
makeTestEndpoints("ns2", "ep2", multipleSubsetsIPsPorts2),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", multipleSubsetsIPsPorts1),
|
||
|
|
makeTestEndpoints("ns2", "ep2", multipleSubsetsIPsPorts2),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.2:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.1:12", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p13"): {
|
||
|
|
{Endpoint: "1.1.1.3:13", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.4:13", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p14"): {
|
||
|
|
{Endpoint: "1.1.1.3:14", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.4:14", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns2", "ep2", "p21"): {
|
||
|
|
{Endpoint: "2.2.2.1:21", IsLocal: false},
|
||
|
|
{Endpoint: "2.2.2.2:21", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns2", "ep2", "p22"): {
|
||
|
|
{Endpoint: "2.2.2.1:22", IsLocal: false},
|
||
|
|
{Endpoint: "2.2.2.2:22", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.2:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.1:12", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p13"): {
|
||
|
|
{Endpoint: "1.1.1.3:13", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.4:13", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p14"): {
|
||
|
|
{Endpoint: "1.1.1.3:14", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.4:14", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns2", "ep2", "p21"): {
|
||
|
|
{Endpoint: "2.2.2.1:21", IsLocal: false},
|
||
|
|
{Endpoint: "2.2.2.2:21", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns2", "ep2", "p22"): {
|
||
|
|
{Endpoint: "2.2.2.1:22", IsLocal: false},
|
||
|
|
{Endpoint: "2.2.2.2:22", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{
|
||
|
|
makeNSN("ns1", "ep1"): 2,
|
||
|
|
makeNSN("ns2", "ep2"): 1,
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[6]: add an Endpoints
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
nil,
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", unnamedPortLocal),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): true,
|
||
|
|
},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{
|
||
|
|
makeNSN("ns1", "ep1"): 1,
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[7]: remove an Endpoints
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", unnamedPortLocal),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
nil,
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{{
|
||
|
|
Endpoint: "1.1.1.1:11",
|
||
|
|
ServicePortName: makeServicePortName("ns1", "ep1", ""),
|
||
|
|
}},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
}, {
|
||
|
|
// Case[8]: add an IP and port
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPort),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPortsLocalNoLocal),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.2:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.1:12", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): true,
|
||
|
|
},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{
|
||
|
|
makeNSN("ns1", "ep1"): 1,
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[9]: remove an IP and port
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPortsLocalNoLocal),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPort),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.2:11", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.1:12", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{{
|
||
|
|
Endpoint: "1.1.1.2:11",
|
||
|
|
ServicePortName: makeServicePortName("ns1", "ep1", "p11"),
|
||
|
|
}, {
|
||
|
|
Endpoint: "1.1.1.1:12",
|
||
|
|
ServicePortName: makeServicePortName("ns1", "ep1", "p12"),
|
||
|
|
}, {
|
||
|
|
Endpoint: "1.1.1.2:12",
|
||
|
|
ServicePortName: makeServicePortName("ns1", "ep1", "p12"),
|
||
|
|
}},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
}, {
|
||
|
|
// Case[10]: add a subset
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPort),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", multipleSubsetsWithLocal),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): true,
|
||
|
|
},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{
|
||
|
|
makeNSN("ns1", "ep1"): 1,
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[11]: remove a subset
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", multipleSubsets),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPort),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{{
|
||
|
|
Endpoint: "1.1.1.2:12",
|
||
|
|
ServicePortName: makeServicePortName("ns1", "ep1", "p12"),
|
||
|
|
}},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
}, {
|
||
|
|
// Case[12]: rename a port
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPort),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPortRenamed),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11-2"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{{
|
||
|
|
Endpoint: "1.1.1.1:11",
|
||
|
|
ServicePortName: makeServicePortName("ns1", "ep1", "p11"),
|
||
|
|
}},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11-2"): true,
|
||
|
|
},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
}, {
|
||
|
|
// Case[13]: renumber a port
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPort),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", namedPortRenumbered),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:22", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{{
|
||
|
|
Endpoint: "1.1.1.1:11",
|
||
|
|
ServicePortName: makeServicePortName("ns1", "ep1", "p11"),
|
||
|
|
}},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
}, {
|
||
|
|
// Case[14]: complex add and remove
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", complexBefore1),
|
||
|
|
makeTestEndpoints("ns2", "ep2", complexBefore2),
|
||
|
|
nil,
|
||
|
|
makeTestEndpoints("ns4", "ep4", complexBefore4),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", complexAfter1),
|
||
|
|
nil,
|
||
|
|
makeTestEndpoints("ns3", "ep3", complexAfter3),
|
||
|
|
makeTestEndpoints("ns4", "ep4", complexAfter4),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns2", "ep2", "p22"): {
|
||
|
|
{Endpoint: "2.2.2.2:22", IsLocal: true},
|
||
|
|
{Endpoint: "2.2.2.22:22", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns2", "ep2", "p23"): {
|
||
|
|
{Endpoint: "2.2.2.3:23", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns4", "ep4", "p44"): {
|
||
|
|
{Endpoint: "4.4.4.4:44", IsLocal: true},
|
||
|
|
{Endpoint: "4.4.4.5:44", IsLocal: true},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns4", "ep4", "p45"): {
|
||
|
|
{Endpoint: "4.4.4.6:45", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", "p11"): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
{Endpoint: "1.1.1.11:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): {
|
||
|
|
{Endpoint: "1.1.1.2:12", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns1", "ep1", "p122"): {
|
||
|
|
{Endpoint: "1.1.1.2:122", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns3", "ep3", "p33"): {
|
||
|
|
{Endpoint: "3.3.3.3:33", IsLocal: false},
|
||
|
|
},
|
||
|
|
makeServicePortName("ns4", "ep4", "p44"): {
|
||
|
|
{Endpoint: "4.4.4.4:44", IsLocal: true},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{{
|
||
|
|
Endpoint: "2.2.2.2:22",
|
||
|
|
ServicePortName: makeServicePortName("ns2", "ep2", "p22"),
|
||
|
|
}, {
|
||
|
|
Endpoint: "2.2.2.22:22",
|
||
|
|
ServicePortName: makeServicePortName("ns2", "ep2", "p22"),
|
||
|
|
}, {
|
||
|
|
Endpoint: "2.2.2.3:23",
|
||
|
|
ServicePortName: makeServicePortName("ns2", "ep2", "p23"),
|
||
|
|
}, {
|
||
|
|
Endpoint: "4.4.4.5:44",
|
||
|
|
ServicePortName: makeServicePortName("ns4", "ep4", "p44"),
|
||
|
|
}, {
|
||
|
|
Endpoint: "4.4.4.6:45",
|
||
|
|
ServicePortName: makeServicePortName("ns4", "ep4", "p45"),
|
||
|
|
}},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{
|
||
|
|
makeServicePortName("ns1", "ep1", "p12"): true,
|
||
|
|
makeServicePortName("ns1", "ep1", "p122"): true,
|
||
|
|
makeServicePortName("ns3", "ep3", "p33"): true,
|
||
|
|
},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{
|
||
|
|
makeNSN("ns4", "ep4"): 1,
|
||
|
|
},
|
||
|
|
}, {
|
||
|
|
// Case[15]: change from 0 endpoint address to 1 unnamed port
|
||
|
|
previousEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", emptyEndpoint),
|
||
|
|
},
|
||
|
|
currentEndpoints: []*api.Endpoints{
|
||
|
|
makeTestEndpoints("ns1", "ep1", unnamedPort),
|
||
|
|
},
|
||
|
|
oldEndpoints: map[ServicePortName][]*BaseEndpointInfo{},
|
||
|
|
expectedResult: map[ServicePortName][]*BaseEndpointInfo{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): {
|
||
|
|
{Endpoint: "1.1.1.1:11", IsLocal: false},
|
||
|
|
},
|
||
|
|
},
|
||
|
|
expectedStaleEndpoints: []ServiceEndpoint{},
|
||
|
|
expectedStaleServiceNames: map[ServicePortName]bool{
|
||
|
|
makeServicePortName("ns1", "ep1", ""): true,
|
||
|
|
},
|
||
|
|
expectedHealthchecks: map[types.NamespacedName]int{},
|
||
|
|
},
|
||
|
|
}
|
||
|
|
|
||
|
|
for tci, tc := range testCases {
|
||
|
|
fp := newFakeProxier()
|
||
|
|
fp.hostname = nodeName
|
||
|
|
|
||
|
|
// First check that after adding all previous versions of endpoints,
|
||
|
|
// the fp.oldEndpoints is as we expect.
|
||
|
|
for i := range tc.previousEndpoints {
|
||
|
|
if tc.previousEndpoints[i] != nil {
|
||
|
|
fp.addEndpoints(tc.previousEndpoints[i])
|
||
|
|
}
|
||
|
|
}
|
||
|
|
UpdateEndpointsMap(fp.endpointsMap, fp.endpointsChanges)
|
||
|
|
compareEndpointsMaps(t, tci, fp.endpointsMap, tc.oldEndpoints)
|
||
|
|
|
||
|
|
// Now let's call appropriate handlers to get to state we want to be.
|
||
|
|
if len(tc.previousEndpoints) != len(tc.currentEndpoints) {
|
||
|
|
t.Fatalf("[%d] different lengths of previous and current endpoints", tci)
|
||
|
|
continue
|
||
|
|
}
|
||
|
|
|
||
|
|
for i := range tc.previousEndpoints {
|
||
|
|
prev, curr := tc.previousEndpoints[i], tc.currentEndpoints[i]
|
||
|
|
switch {
|
||
|
|
case prev == nil:
|
||
|
|
fp.addEndpoints(curr)
|
||
|
|
case curr == nil:
|
||
|
|
fp.deleteEndpoints(prev)
|
||
|
|
default:
|
||
|
|
fp.updateEndpoints(prev, curr)
|
||
|
|
}
|
||
|
|
}
|
||
|
|
result := UpdateEndpointsMap(fp.endpointsMap, fp.endpointsChanges)
|
||
|
|
newMap := fp.endpointsMap
|
||
|
|
compareEndpointsMaps(t, tci, newMap, tc.expectedResult)
|
||
|
|
if len(result.StaleEndpoints) != len(tc.expectedStaleEndpoints) {
|
||
|
|
t.Errorf("[%d] expected %d staleEndpoints, got %d: %v", tci, len(tc.expectedStaleEndpoints), len(result.StaleEndpoints), result.StaleEndpoints)
|
||
|
|
}
|
||
|
|
for _, x := range tc.expectedStaleEndpoints {
|
||
|
|
found := false
|
||
|
|
for _, stale := range result.StaleEndpoints {
|
||
|
|
if stale == x {
|
||
|
|
found = true
|
||
|
|
break
|
||
|
|
}
|
||
|
|
}
|
||
|
|
if !found {
|
||
|
|
t.Errorf("[%d] expected staleEndpoints[%v], but didn't find it: %v", tci, x, result.StaleEndpoints)
|
||
|
|
}
|
||
|
|
}
|
||
|
|
if len(result.StaleServiceNames) != len(tc.expectedStaleServiceNames) {
|
||
|
|
t.Errorf("[%d] expected %d staleServiceNames, got %d: %v", tci, len(tc.expectedStaleServiceNames), len(result.StaleServiceNames), result.StaleServiceNames)
|
||
|
|
}
|
||
|
|
for svcName := range tc.expectedStaleServiceNames {
|
||
|
|
found := false
|
||
|
|
for _, stale := range result.StaleServiceNames {
|
||
|
|
if stale == svcName {
|
||
|
|
found = true
|
||
|
|
}
|
||
|
|
}
|
||
|
|
if !found {
|
||
|
|
t.Errorf("[%d] expected staleServiceNames[%v], but didn't find it: %v", tci, svcName, result.StaleServiceNames)
|
||
|
|
}
|
||
|
|
}
|
||
|
|
if !reflect.DeepEqual(result.HCEndpointsLocalIPSize, tc.expectedHealthchecks) {
|
||
|
|
t.Errorf("[%d] expected healthchecks %v, got %v", tci, tc.expectedHealthchecks, result.HCEndpointsLocalIPSize)
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|
||
|
|
|
||
|
|
func compareEndpointsMaps(t *testing.T, tci int, newMap EndpointsMap, expected map[ServicePortName][]*BaseEndpointInfo) {
|
||
|
|
if len(newMap) != len(expected) {
|
||
|
|
t.Errorf("[%d] expected %d results, got %d: %v", tci, len(expected), len(newMap), newMap)
|
||
|
|
}
|
||
|
|
for x := range expected {
|
||
|
|
if len(newMap[x]) != len(expected[x]) {
|
||
|
|
t.Errorf("[%d] expected %d endpoints for %v, got %d", tci, len(expected[x]), x, len(newMap[x]))
|
||
|
|
} else {
|
||
|
|
for i := range expected[x] {
|
||
|
|
newEp, ok := newMap[x][i].(*BaseEndpointInfo)
|
||
|
|
if !ok {
|
||
|
|
t.Errorf("Failed to cast endpointsInfo")
|
||
|
|
continue
|
||
|
|
}
|
||
|
|
if *newEp != *(expected[x][i]) {
|
||
|
|
t.Errorf("[%d] expected new[%v][%d] to be %v, got %v", tci, x, i, expected[x][i], newEp)
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|
||
|
|
}
|