mirrors/external-dns
1
0
Fork 0
mirror of https://github.com/kubernetes-sigs/external-dns.git synced 2025-08-05 17:16:59 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix(idna): fix handling of domains

Browse Source 
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
This commit is contained in:
ivan katliarchuk 2025-07-25 22:17:14 +01:00
parent e9ec3acdaa
commit 474d5b3600
No known key found for this signature in database
GPG Key ID: 601CDBBBB76E47BE
1 changed files with 39 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

79
provider/aws/aws.go
View File

@ -734,58 +734,57 @@ func (p *AWSProvider) submitChanges(ctx context.Context, changes Route53Changes,
log.Infof("Desired change: %s %s %s", c.Action, *c.ResourceRecordSet.Name, c.ResourceRecordSet.Type) log.Infof("Desired change: %s %s %s", c.Action, *c.ResourceRecordSet.Name, c.ResourceRecordSet.Type)
} }
if p.dryRun { if !p.dryRun {
continue
}
params := &route53.ChangeResourceRecordSetsInput{ params := &route53.ChangeResourceRecordSetsInput{
HostedZoneId: aws.String(z), HostedZoneId: aws.String(z),
ChangeBatch: &route53types.ChangeBatch{ ChangeBatch: &route53types.ChangeBatch{
Changes: b.Route53Changes(), Changes: b.Route53Changes(),
}, },
} }
successfulChanges := 0 successfulChanges := 0
client := p.clients[zones[z].profile] client := p.clients[zones[z].profile]
if _, err := client.ChangeResourceRecordSets(ctx, params); err != nil { if _, err := client.ChangeResourceRecordSets(ctx, params); err != nil {
log.Errorf("Failure in zone %s when submitting change batch: %v", *zones[z].zone.Name, err) log.Errorf("Failure in zone %s when submitting change batch: %v", *zones[z].zone.Name, err)
changesByOwnership := groupChangesByNameAndOwnershipRelation(b) changesByOwnership := groupChangesByNameAndOwnershipRelation(b)
if len(changesByOwnership) > 1 { if len(changesByOwnership) > 1 {
log.Debug("Trying to submit change sets one-by-one instead") log.Debug("Trying to submit change sets one-by-one instead")
for _, changes := range changesByOwnership { for _, changes := range changesByOwnership {
if log.Logger.IsLevelEnabled(debugLevel) { if log.Logger.IsLevelEnabled(debugLevel) {
for _, c := range changes { for _, c := range changes {
log.Debugf("Desired change: %s %s %s", c.Action, *c.ResourceRecordSet.Name, c.ResourceRecordSet.Type) log.Debugf("Desired change: %s %s %s", c.Action, *c.ResourceRecordSet.Name, c.ResourceRecordSet.Type)
}
}
params.ChangeBatch = &route53types.ChangeBatch{
Changes: changes.Route53Changes(),
}
if _, err := client.ChangeResourceRecordSets(ctx, params); err != nil {
failedUpdate = true
log.Errorf("Failed submitting change (error: %v), it will be retried in a separate change batch in the next iteration", err)
p.failedChangesQueue[z] = append(p.failedChangesQueue[z], changes...)
} else {
successfulChanges = successfulChanges + len(changes)
} }
} }
params.ChangeBatch = &route53types.ChangeBatch{ } else {
Changes: changes.Route53Changes(), failedUpdate = true
}
if _, err := client.ChangeResourceRecordSets(ctx, params); err != nil {
failedUpdate = true
log.Errorf("Failed submitting change (error: %v), it will be retried in a separate change batch in the next iteration", err)
p.failedChangesQueue[z] = append(p.failedChangesQueue[z], changes...)
} else {
successfulChanges = successfulChanges + len(changes)
}
} }
} else { } else {
failedUpdate = true successfulChanges = len(b)
} }
} else {
successfulChanges = len(b)
}
if successfulChanges > 0 { if successfulChanges > 0 {
// z is the R53 Hosted Zone ID already as aws.StringValue // z is the R53 Hosted Zone ID already as aws.StringValue
log.Infof("%d record(s) were successfully updated", successfulChanges) log.Infof("%d record(s) were successfully updated", successfulChanges)
} }
if i != len(batchCs)-1 { if i != len(batchCs)-1 {
time.Sleep(p.batchChangeInterval) time.Sleep(p.batchChangeInterval)
}
} }
} }