mirror of
				https://github.com/kubernetes-sigs/external-dns.git
				synced 2025-11-04 04:31:00 +01:00 
			
		
		
		
	* Endpoint.Target is now Endpoint.Targets. This is its own type representing mutliple targets for a single DNS name while adding some convenience for sorting and comparing * Made everything compile and tests run through with the new Endpoint.Targets * The ingress source can now properly handle multiple target ips per host * Added custom conflict resolver, to better understand how conflict resolution has to work for me * My custom conflict resolver behaves a bit different than the PerResource resolver, therefore I needed to modify the expected test result Removed unnecessary FIXME * The ingress source now creates CNAME endpoints with multiple targets to let the DNS provider decide how to handle multiple CNAME targets. This could be interesting for weighted targets etc. * Adopted the expected results to the new way we create endpoints for CNAMEs * Removed Add method from Targets since manipulating the slice through here is unnecessary complicated and doesn't deliver enough convenience * Reverted ConflictResolver to the original one. There is some discussing to do what the best way is to handle conflicts * Added missing documenting comment to IsLess of Targets * Added documenting comments to Targets,Targets.Same and NewTargets to clarify their intention and usage * Service source now also generates endpoints with multiple targets * Service and Ingress source now sort all Targets for every Endpoint to make order of Targets predictable * Endpoints generated by the Google Cloud DNS provider now also have sorted Targets to make order of Targets predictable * Modified provider dyn to be able to compile with multi target changes * Fixed small nitpicks, so my code is acceptable * Fixed merge method after updating to new Targets. Replacing '!=' with .Same of course needs a boolean negation * Tests for dyn provider now also use the new Targets instead of Target * Simplified extractServiceIps as implied by linki to make it more readable * ref: change service ClusterIP retrieval again * Added entry to CHANGELOG.md describing the new features contained in this PR
		
			
				
	
	
		
			113 lines
		
	
	
		
			3.5 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			113 lines
		
	
	
		
			3.5 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 source
 | 
						|
 | 
						|
import (
 | 
						|
	"testing"
 | 
						|
 | 
						|
	"github.com/kubernetes-incubator/external-dns/endpoint"
 | 
						|
	"github.com/kubernetes-incubator/external-dns/internal/testutils"
 | 
						|
)
 | 
						|
 | 
						|
// Validates that dedupSource is a Source
 | 
						|
var _ Source = &dedupSource{}
 | 
						|
 | 
						|
func TestDedup(t *testing.T) {
 | 
						|
	t.Run("Endpoints", testDedupEndpoints)
 | 
						|
}
 | 
						|
 | 
						|
// testDedupEndpoints tests that duplicates from the wrapped source are removed.
 | 
						|
func testDedupEndpoints(t *testing.T) {
 | 
						|
	for _, tc := range []struct {
 | 
						|
		title     string
 | 
						|
		endpoints []*endpoint.Endpoint
 | 
						|
		expected  []*endpoint.Endpoint
 | 
						|
	}{
 | 
						|
		{
 | 
						|
			"one endpoint returns one endpoint",
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
			},
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
			},
 | 
						|
		},
 | 
						|
		{
 | 
						|
			"two different endpoints return two endpoints",
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
				{DNSName: "bar.example.org", Targets: endpoint.Targets{"4.5.6.7"}},
 | 
						|
			},
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
				{DNSName: "bar.example.org", Targets: endpoint.Targets{"4.5.6.7"}},
 | 
						|
			},
 | 
						|
		},
 | 
						|
		{
 | 
						|
			"two endpoints with same dnsname and different targets return two endpoints",
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"4.5.6.7"}},
 | 
						|
			},
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"4.5.6.7"}},
 | 
						|
			},
 | 
						|
		},
 | 
						|
		{
 | 
						|
			"two endpoints with different dnsname and same target return two endpoints",
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
				{DNSName: "bar.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
			},
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
				{DNSName: "bar.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
			},
 | 
						|
		},
 | 
						|
		{
 | 
						|
			"two endpoints with same dnsname and same target return one endpoint",
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
			},
 | 
						|
			[]*endpoint.Endpoint{
 | 
						|
				{DNSName: "foo.example.org", Targets: endpoint.Targets{"1.2.3.4"}},
 | 
						|
			},
 | 
						|
		},
 | 
						|
	} {
 | 
						|
		t.Run(tc.title, func(t *testing.T) {
 | 
						|
			mockSource := new(testutils.MockSource)
 | 
						|
			mockSource.On("Endpoints").Return(tc.endpoints, nil)
 | 
						|
 | 
						|
			// Create our object under test and get the endpoints.
 | 
						|
			source := NewDedupSource(mockSource)
 | 
						|
 | 
						|
			endpoints, err := source.Endpoints()
 | 
						|
			if err != nil {
 | 
						|
				t.Fatal(err)
 | 
						|
			}
 | 
						|
 | 
						|
			// Validate returned endpoints against desired endpoints.
 | 
						|
			validateEndpoints(t, endpoints, tc.expected)
 | 
						|
 | 
						|
			// Validate that the mock source was called.
 | 
						|
			mockSource.AssertExpectations(t)
 | 
						|
		})
 | 
						|
	}
 | 
						|
}
 |