diff --git a/upup/pkg/fi/cloudup/dns_test.go b/upup/pkg/fi/cloudup/dns_test.go index 2df0088d8a..9151289d4b 100644 --- a/upup/pkg/fi/cloudup/dns_test.go +++ b/upup/pkg/fi/cloudup/dns_test.go @@ -30,13 +30,64 @@ func TestPrecreateDNSNames(t *testing.T) { cluster *kops.Cluster expected []recordKey }{ + { + cluster: &kops.Cluster{ + Spec: kops.ClusterSpec{ + API: kops.APISpec{ + LoadBalancer: &kops.LoadBalancerAccessSpec{}, + }, + CloudProvider: kops.CloudProviderSpec{ + AWS: &kops.AWSSpec{}, + }, + }, + }, + expected: []recordKey{ + {"api.internal.cluster1.example.com", rrstype.A}, + {"kops-controller.internal.cluster1.example.com", rrstype.A}, + }, + }, + { + cluster: &kops.Cluster{ + Spec: kops.ClusterSpec{ + API: kops.APISpec{ + LoadBalancer: &kops.LoadBalancerAccessSpec{}, + }, + CloudProvider: kops.CloudProviderSpec{ + AWS: &kops.AWSSpec{}, + }, + Networking: kops.NetworkingSpec{ + NonMasqueradeCIDR: "::/0", + }, + }, + }, + expected: []recordKey{ + {"api.internal.cluster1.example.com", rrstype.AAAA}, + {"kops-controller.internal.cluster1.example.com", rrstype.AAAA}, + }, + }, + { + cluster: &kops.Cluster{ + Spec: kops.ClusterSpec{ + API: kops.APISpec{ + LoadBalancer: &kops.LoadBalancerAccessSpec{ + UseForInternalAPI: true, + }, + }, + CloudProvider: kops.CloudProviderSpec{ + AWS: &kops.AWSSpec{}, + }, + }, + }, + expected: []recordKey{ + {"kops-controller.internal.cluster1.example.com", rrstype.A}, + }, + }, { cluster: &kops.Cluster{ Spec: kops.ClusterSpec{ CloudProvider: kops.CloudProviderSpec{ AWS: &kops.AWSSpec{}, }, - KubernetesVersion: "1.22.0", }, }, expected: []recordKey{ @@ -51,7 +102,6 @@ func TestPrecreateDNSNames(t *testing.T) { CloudProvider: kops.CloudProviderSpec{ AWS: &kops.AWSSpec{}, }, - KubernetesVersion: "1.22.0", Networking: kops.NetworkingSpec{ NonMasqueradeCIDR: "::/0", },