fix(linode): don't try to create existing records

A bug in the Linode provider can result in an unbounded explosion of TXT records being created.
For example, if external-dns attempts to create a CNAME in a zone that has a conflicting A record,
it will fail but will still create 2 new TXT records every single time.

Instead, we can just skip creating records that already exist since they should
only ever be in the list of updates.
This commit is contained in:
Will Hegedus 2023-08-31 19:18:55 -04:00
parent f17e24ff8a
commit a2fedbc1dd

View File

@ -315,7 +315,8 @@ func (p *LinodeProvider) ApplyChanges(ctx context.Context, changes *plan.Changes
"zoneName": zone.Domain,
"dnsName": ep.DNSName,
"recordType": ep.RecordType,
}).Warn("Records found which should not exist")
}).Warn("Records found which should not exist. Not touching it.")
continue
}
recordType, err := convertRecordType(ep.RecordType)