mirror of
https://github.com/kubernetes-sigs/external-dns.git
synced 2025-08-05 17:16:59 +02:00
* feat(txt-registry): only support single format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): only support single format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): only support single format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): only support single format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): only support single format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com> * feat(txt-registry): deprecate legacy txt-format Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): address review comments Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format * feat(txt-registry): deprecate legacy txt-format Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> * feat(txt-registry): deprecate legacy txt-format Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> --------- Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com> Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>
498 lines
12 KiB
Go
498 lines
12 KiB
Go
/*
|
|
Copyright 2025 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 controller
|
|
|
|
import (
|
|
"bytes"
|
|
"context"
|
|
"fmt"
|
|
"net"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"os"
|
|
"os/signal"
|
|
"reflect"
|
|
"regexp"
|
|
"syscall"
|
|
"testing"
|
|
"time"
|
|
|
|
log "github.com/sirupsen/logrus"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
"sigs.k8s.io/external-dns/endpoint"
|
|
"sigs.k8s.io/external-dns/pkg/apis/externaldns"
|
|
"sigs.k8s.io/external-dns/plan"
|
|
"sigs.k8s.io/external-dns/provider"
|
|
)
|
|
|
|
func TestSelectRegistry(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg *externaldns.Config
|
|
provider provider.Provider
|
|
wantErr bool
|
|
wantType string
|
|
}{
|
|
{
|
|
name: "DynamoDB registry",
|
|
cfg: &externaldns.Config{
|
|
Registry: "dynamodb",
|
|
AWSDynamoDBRegion: "us-west-2",
|
|
AWSDynamoDBTable: "test-table",
|
|
TXTOwnerID: "owner-id",
|
|
TXTWildcardReplacement: "wildcard",
|
|
ManagedDNSRecordTypes: []string{"A", "CNAME"},
|
|
ExcludeDNSRecordTypes: []string{"TXT"},
|
|
TXTCacheInterval: 60,
|
|
},
|
|
provider: &MockProvider{},
|
|
wantErr: false,
|
|
wantType: "DynamoDBRegistry",
|
|
},
|
|
{
|
|
name: "Noop registry",
|
|
cfg: &externaldns.Config{
|
|
Registry: "noop",
|
|
},
|
|
provider: &MockProvider{},
|
|
wantErr: false,
|
|
wantType: "NoopRegistry",
|
|
},
|
|
{
|
|
name: "TXT registry",
|
|
cfg: &externaldns.Config{
|
|
Registry: "txt",
|
|
TXTPrefix: "prefix",
|
|
TXTOwnerID: "owner-id",
|
|
TXTCacheInterval: 60,
|
|
TXTWildcardReplacement: "wildcard",
|
|
ManagedDNSRecordTypes: []string{"A", "CNAME"},
|
|
ExcludeDNSRecordTypes: []string{"TXT"},
|
|
},
|
|
provider: &MockProvider{},
|
|
wantErr: false,
|
|
wantType: "TXTRegistry",
|
|
},
|
|
{
|
|
name: "AWS-SD registry",
|
|
cfg: &externaldns.Config{
|
|
Registry: "aws-sd",
|
|
TXTOwnerID: "owner-id",
|
|
},
|
|
provider: &MockProvider{},
|
|
wantErr: false,
|
|
wantType: "AWSSDRegistry",
|
|
},
|
|
{
|
|
name: "Unknown registry",
|
|
cfg: &externaldns.Config{
|
|
Registry: "unknown",
|
|
},
|
|
provider: &MockProvider{},
|
|
wantErr: true,
|
|
wantType: "",
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
if tt.wantErr {
|
|
defer func() { log.StandardLogger().ExitFunc = nil }()
|
|
b := new(bytes.Buffer)
|
|
log.StandardLogger().ExitFunc = func(int) {}
|
|
log.StandardLogger().SetOutput(b)
|
|
|
|
_, err := selectRegistry(tt.cfg, tt.provider)
|
|
assert.NoError(t, err)
|
|
assert.Contains(t, b.String(), "unknown registry: unknown")
|
|
} else {
|
|
reg, err := selectRegistry(tt.cfg, tt.provider)
|
|
assert.NoError(t, err)
|
|
assert.Contains(t, reflect.TypeOf(reg).String(), tt.wantType)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestCreateDomainFilter(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg *externaldns.Config
|
|
expectedDomainFilter *endpoint.DomainFilter
|
|
isConfigured bool
|
|
}{
|
|
{
|
|
name: "RegexDomainFilter",
|
|
cfg: &externaldns.Config{
|
|
RegexDomainFilter: regexp.MustCompile(`example\.com`),
|
|
RegexDomainExclusion: regexp.MustCompile(`excluded\.example\.com`),
|
|
},
|
|
expectedDomainFilter: endpoint.NewRegexDomainFilter(regexp.MustCompile(`example\.com`), regexp.MustCompile(`excluded\.example\.com`)),
|
|
isConfigured: true,
|
|
},
|
|
{
|
|
name: "RegexDomainWithoutExclusionFilter",
|
|
cfg: &externaldns.Config{
|
|
RegexDomainFilter: regexp.MustCompile(`example\.com`),
|
|
},
|
|
expectedDomainFilter: endpoint.NewRegexDomainFilter(regexp.MustCompile(`example\.com`), nil),
|
|
isConfigured: true,
|
|
},
|
|
{
|
|
name: "DomainFilterWithExclusions",
|
|
cfg: &externaldns.Config{
|
|
DomainFilter: []string{"example.com"},
|
|
ExcludeDomains: []string{"excluded.example.com"},
|
|
},
|
|
expectedDomainFilter: endpoint.NewDomainFilterWithExclusions([]string{"example.com"}, []string{"excluded.example.com"}),
|
|
isConfigured: true,
|
|
},
|
|
{
|
|
name: "DomainFilterWithExclusionsOnly",
|
|
cfg: &externaldns.Config{
|
|
ExcludeDomains: []string{"excluded.example.com"},
|
|
},
|
|
expectedDomainFilter: endpoint.NewDomainFilterWithExclusions([]string{}, []string{"excluded.example.com"}),
|
|
isConfigured: true,
|
|
},
|
|
{
|
|
name: "EmptyDomainFilter",
|
|
cfg: &externaldns.Config{
|
|
DomainFilter: []string{},
|
|
ExcludeDomains: []string{},
|
|
},
|
|
expectedDomainFilter: endpoint.NewDomainFilterWithExclusions([]string{}, []string{}),
|
|
isConfigured: false,
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
filter := createDomainFilter(tt.cfg)
|
|
assert.Equal(t, tt.isConfigured, filter.IsConfigured())
|
|
assert.Equal(t, tt.expectedDomainFilter, filter)
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestHandleSigterm(t *testing.T) {
|
|
cancelCalled := make(chan bool, 1)
|
|
cancel := func() {
|
|
cancelCalled <- true
|
|
}
|
|
|
|
var logOutput bytes.Buffer
|
|
log.SetOutput(&logOutput)
|
|
defer log.SetOutput(os.Stderr)
|
|
|
|
go handleSigterm(cancel)
|
|
|
|
// Simulate sending a SIGTERM signal
|
|
sigChan := make(chan os.Signal, 1)
|
|
signal.Notify(sigChan, syscall.SIGTERM)
|
|
err := syscall.Kill(syscall.Getpid(), syscall.SIGTERM)
|
|
assert.NoError(t, err)
|
|
|
|
// Wait for the cancel function to be called
|
|
select {
|
|
case <-cancelCalled:
|
|
assert.Contains(t, logOutput.String(), "Received SIGTERM. Terminating...")
|
|
case sig := <-sigChan:
|
|
assert.Equal(t, syscall.SIGTERM, sig)
|
|
case <-time.After(1 * time.Second):
|
|
t.Fatal("cancel function was not called")
|
|
}
|
|
}
|
|
|
|
func getRandomPort() (int, error) {
|
|
addr, err := net.ResolveTCPAddr("tcp", "localhost:0")
|
|
if err != nil {
|
|
return 0, err
|
|
}
|
|
|
|
l, err := net.ListenTCP("tcp", addr)
|
|
if err != nil {
|
|
return 0, err
|
|
}
|
|
defer l.Close()
|
|
return l.Addr().(*net.TCPAddr).Port, nil
|
|
}
|
|
|
|
func TestServeMetrics(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
port, err := getRandomPort()
|
|
require.NoError(t, err)
|
|
addresse := fmt.Sprintf("localhost:%d", port)
|
|
|
|
go serveMetrics(fmt.Sprintf(":%d", port))
|
|
|
|
// Wait for the TCP socket to be ready
|
|
require.Eventually(t, func() bool {
|
|
conn, err := net.Dial("tcp", addresse)
|
|
if err != nil {
|
|
return false
|
|
}
|
|
_ = conn.Close()
|
|
return true
|
|
}, 1*time.Second, 5*time.Millisecond, "server not ready with port open in time")
|
|
|
|
resp, err := http.Get(fmt.Sprintf("http://%s/healthz", addresse))
|
|
require.NoError(t, err)
|
|
assert.Equal(t, http.StatusOK, resp.StatusCode)
|
|
|
|
resp, err = http.Get(fmt.Sprintf("http://%s/metrics", addresse))
|
|
require.NoError(t, err)
|
|
assert.Equal(t, http.StatusOK, resp.StatusCode)
|
|
}
|
|
|
|
func TestConfigureLogger(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg *externaldns.Config
|
|
wantLevel log.Level
|
|
wantJSON bool
|
|
wantErr bool
|
|
wantErrMsg string
|
|
}{
|
|
{
|
|
name: "Default log format and level",
|
|
cfg: &externaldns.Config{
|
|
LogLevel: "info",
|
|
LogFormat: "text",
|
|
},
|
|
wantLevel: log.InfoLevel,
|
|
},
|
|
{
|
|
name: "JSON log format",
|
|
cfg: &externaldns.Config{
|
|
LogLevel: "debug",
|
|
LogFormat: "json",
|
|
},
|
|
wantLevel: log.DebugLevel,
|
|
wantJSON: true,
|
|
},
|
|
{
|
|
name: "Invalid log level",
|
|
cfg: &externaldns.Config{
|
|
LogLevel: "invalid",
|
|
LogFormat: "text",
|
|
},
|
|
wantLevel: log.InfoLevel,
|
|
wantErr: true,
|
|
wantErrMsg: "failed to parse log level",
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
if tt.wantErr {
|
|
defer func() { log.StandardLogger().ExitFunc = nil }()
|
|
|
|
b := new(bytes.Buffer)
|
|
var captureLogFatal bool
|
|
log.StandardLogger().ExitFunc = func(int) { captureLogFatal = true }
|
|
log.StandardLogger().SetOutput(b)
|
|
|
|
configureLogger(tt.cfg)
|
|
|
|
assert.True(t, captureLogFatal)
|
|
assert.Contains(t, b.String(), tt.wantErrMsg)
|
|
} else {
|
|
configureLogger(tt.cfg)
|
|
assert.Equal(t, tt.wantLevel, log.GetLevel())
|
|
|
|
if tt.wantJSON {
|
|
assert.IsType(t, &log.JSONFormatter{}, log.StandardLogger().Formatter)
|
|
} else {
|
|
assert.IsType(t, &log.TextFormatter{}, log.StandardLogger().Formatter)
|
|
}
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBuildProvider(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg *externaldns.Config
|
|
expectedType string
|
|
expectedError string
|
|
}{
|
|
{
|
|
name: "aws provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "aws",
|
|
},
|
|
expectedType: "*aws.AWSProvider",
|
|
},
|
|
{
|
|
name: "rfc2136 provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "rfc2136",
|
|
RFC2136TSIGSecretAlg: "hmac-sha256",
|
|
},
|
|
expectedType: "*rfc2136.rfc2136Provider",
|
|
},
|
|
{
|
|
name: "gandi provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "gandi",
|
|
},
|
|
expectedError: "no environment variable GANDI_KEY or GANDI_PAT provided",
|
|
},
|
|
{
|
|
name: "inmemory provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "inmemory",
|
|
},
|
|
expectedType: "*inmemory.InMemoryProvider",
|
|
},
|
|
{
|
|
name: "inmemory cached provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "inmemory",
|
|
ProviderCacheTime: 10 * time.Millisecond,
|
|
},
|
|
expectedType: "*provider.CachedProvider",
|
|
},
|
|
{
|
|
name: "coredns provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "coredns",
|
|
},
|
|
expectedType: "coredns.coreDNSProvider",
|
|
},
|
|
{
|
|
name: "pihole provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "pihole",
|
|
PiholeApiVersion: "6",
|
|
PiholeServer: "http://localhost:8080",
|
|
},
|
|
expectedType: "*pihole.PiholeProvider",
|
|
},
|
|
{
|
|
name: "dnsimple provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "dnsimple",
|
|
},
|
|
expectedError: "no dnsimple oauth token provided",
|
|
},
|
|
{
|
|
name: "unknown provider",
|
|
cfg: &externaldns.Config{
|
|
Provider: "unknown",
|
|
},
|
|
expectedError: "unknown dns provider: unknown",
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
domainFilter := endpoint.NewDomainFilter([]string{"example.com"})
|
|
|
|
p, err := buildProvider(t.Context(), tt.cfg, domainFilter)
|
|
|
|
if tt.expectedError != "" {
|
|
assert.Error(t, err)
|
|
assert.EqualError(t, err, tt.expectedError)
|
|
} else {
|
|
assert.NoError(t, err)
|
|
assert.NotNil(t, p)
|
|
assert.Equal(t, tt.expectedType, reflect.TypeOf(p).String())
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBuildSource(t *testing.T) {
|
|
svr := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
w.WriteHeader(http.StatusNotImplemented)
|
|
}))
|
|
defer svr.Close()
|
|
|
|
tests := []struct {
|
|
name string
|
|
cfg *externaldns.Config
|
|
expectedError bool
|
|
}{
|
|
{
|
|
name: "Valid configuration with sources",
|
|
cfg: &externaldns.Config{
|
|
APIServerURL: svr.URL,
|
|
Sources: []string{"fake"},
|
|
RequestTimeout: 6 * time.Millisecond,
|
|
},
|
|
expectedError: false,
|
|
},
|
|
{
|
|
name: "Empty sources configuration",
|
|
cfg: &externaldns.Config{
|
|
APIServerURL: svr.URL,
|
|
Sources: []string{},
|
|
RequestTimeout: 6 * time.Millisecond,
|
|
},
|
|
expectedError: false,
|
|
},
|
|
{
|
|
name: "Update events enabled",
|
|
cfg: &externaldns.Config{
|
|
KubeConfig: "path-to-kubeconfig-not-exists",
|
|
APIServerURL: svr.URL,
|
|
Sources: []string{"ingress"},
|
|
UpdateEvents: true,
|
|
},
|
|
expectedError: true,
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
src, err := buildSource(t.Context(), tt.cfg)
|
|
|
|
if tt.expectedError {
|
|
assert.Error(t, err)
|
|
assert.Nil(t, src)
|
|
} else {
|
|
require.NoError(t, err)
|
|
assert.NotNil(t, src)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
// mocks
|
|
type MockProvider struct{}
|
|
|
|
func (m *MockProvider) Records(ctx context.Context) ([]*endpoint.Endpoint, error) {
|
|
return nil, nil
|
|
}
|
|
|
|
func (p *MockProvider) ApplyChanges(ctx context.Context, changes *plan.Changes) error {
|
|
return nil
|
|
}
|
|
|
|
func (m *MockProvider) AdjustEndpoints(endpoints []*endpoint.Endpoint) ([]*endpoint.Endpoint, error) {
|
|
return nil, nil
|
|
}
|
|
|
|
func (m *MockProvider) GetDomainFilter() endpoint.DomainFilterInterface {
|
|
return nil
|
|
}
|