kubernetes-sigs / external-dns

Configure external DNS servers (AWS Route53, Google CloudDNS and others) for Kubernetes Ingresses and Services
Apache License 2.0
7.62k stars 2.55k forks source link

TXT registry: only handle forward migration #4358

Open vmax opened 6 months ago

vmax commented 6 months ago

What happened:

I'm developing a webhook provider. Underlying DNS provider rejects trying to create CNAME and TXT record of the same name (which abides by the RFC AFAIR). However I see that it attempts to create both old-style and new-style DNS TXT records:

New-style records are created just fine (as they don't conflict with CNAME).

However, on the next run it still tries to create old-style records (as txt/force-update: true provider-specific property has been assigned)

What you expected to happen:

Registry is constructed from new-style records only; absence of old records is discarded

How to reproduce it (as minimally and precisely as possible):

WIP

Anything else we need to know?:

I believe the fix would be to either:

Environment:

k8s-triage-robot commented 3 months ago

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

You can:

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

vmax commented 3 months ago

/remove-lifecycle stale

k8s-triage-robot commented 4 days ago

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

You can:

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

vmax commented 4 days ago

/remove-lifecycle stale