diff --git a/pkg/controllers/ingress_controller.go b/pkg/controllers/ingress_controller.go index c2a2939..f85572c 100644 --- a/pkg/controllers/ingress_controller.go +++ b/pkg/controllers/ingress_controller.go @@ -175,8 +175,6 @@ func (r *IngressReconciler) newDnsEndpoint(ctx context.Context, dnsEndpoint *ext setOwnerRef(dnsEndpoint, &ingress) - dnsEndpoint.Name = ingress.ObjectMeta.Name - dnsEndpoint.Namespace = ingress.ObjectMeta.Namespace desiredWeight = uint(trafficweight.Store.DesiredWeight) if r.isIngressWeighted(ingress) { desiredWeight, err = r.calculateIngressWeight(ingress) diff --git a/pkg/controllers/ingress_controller_test.go b/pkg/controllers/ingress_controller_test.go index 8d647c0..947df63 100644 --- a/pkg/controllers/ingress_controller_test.go +++ b/pkg/controllers/ingress_controller_test.go @@ -130,7 +130,12 @@ func TestIngressController(t *testing.T) { } t.Run("Should forge DNS Endpoints", func(t *testing.T) { - forged := &externaldnsk8siov1alpha1.DNSEndpoint{} + forged := &externaldnsk8siov1alpha1.DNSEndpoint{ + ObjectMeta: metav1.ObjectMeta{ + Name: "bar", + Namespace: "foo", + }, + } reconciler.newDnsEndpoint(context.Background(), forged, ing) assert.Equal(t, expected, *forged) }) @@ -194,7 +199,12 @@ func TestIngressController(t *testing.T) { t.Run("if we dont set --aws-health-check-id ingress shouldnt have health property", func(t *testing.T) { trafficweight.Store.AWSHealthCheckID = "" - forged := &externaldnsk8siov1alpha1.DNSEndpoint{} + forged := &externaldnsk8siov1alpha1.DNSEndpoint{ + ObjectMeta: metav1.ObjectMeta{ + Name: "bar", + Namespace: "foo", + }, + } reconciler.newDnsEndpoint(context.Background(), forged, ing) assert.Equal(t, expected, *forged) }) @@ -229,7 +239,12 @@ func TestIngressController(t *testing.T) { }, } - forged := &externaldnsk8siov1alpha1.DNSEndpoint{} + forged := &externaldnsk8siov1alpha1.DNSEndpoint{ + ObjectMeta: metav1.ObjectMeta{ + Name: "bar", + Namespace: "foo", + }, + } reconciler.newDnsEndpoint(context.Background(), forged, ing) assert.Equal(t, expected, *forged) ing.Spec.Rules = oldRules