external-dns icon indicating copy to clipboard operation
external-dns copied to clipboard

AWS Route53 provider doesn't appear to obey domain exclusion filters when loading Zones

Open willhughes-au opened this issue 2 years ago • 7 comments
trafficstars

What happened:

I have an AWS Account with two Route53 zones:

  • example.com
  • subdomain.example.com

When I run external-dns with --domain-filter=example.com it also picks up the zone subdomain.example.com, and I can see the following output from external-dns:

Applying provider record filter for domains: [example.com. .example.com. subdomain.example.com. .subdomain.example.com.]

When I then pass in --exclude-domains=subdomain.example.com, I get the same output.

What you expected to happen:

I expect that --exclude-domains (and the other filtering options as documented) would apply to the Zones method, and those Zones should not be returned.

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

Have an AWS Account with at least two Route53 zones, with one zone being a subdomain of another.

Anything else we need to know?:

Environment:

  • External-DNS version (use external-dns --version): Running from container registry.k8s.io/external-dns/external-dns:v0.13.5
  • DNS provider: AWS Route53
  • Others:

willhughes-au avatar Jun 21 '23 01:06 willhughes-au

Hi,

I'm running 0.13.1 and its working fine.

Helm chart expects it as "extraArgs":

extraArgs:
  - --exclude-domains=xxx
  - --exclude-domains=xxx

rkferreira avatar Jun 21 '23 13:06 rkferreira

Yeah, I am providing it as extraArgs in the chart.

I can see the arguments being passed to the pod. That's not the issue.

It's definitely ignoring it for me on 0.13.5.

willhughes-au avatar Jun 26 '23 05:06 willhughes-au

Could this be a manifestation of https://github.com/kubernetes-sigs/external-dns/issues/3753 and https://github.com/kubernetes-sigs/external-dns/issues/3948?

joaocc avatar Sep 27 '23 09:09 joaocc

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

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

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

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

/lifecycle stale

k8s-triage-robot avatar Jan 29 '24 06:01 k8s-triage-robot

/remove-lifecycle stale

joaocc avatar Jan 29 '24 15:01 joaocc

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

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

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

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

/lifecycle stale

k8s-triage-robot avatar Apr 28 '24 15:04 k8s-triage-robot

/remove-lifecycle stale

joaocc avatar Apr 29 '24 06:04 joaocc

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

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

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

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

/lifecycle stale

k8s-triage-robot avatar Jul 28 '24 07:07 k8s-triage-robot

/remove-lifecycle stale

joaocc avatar Jul 30 '24 09:07 joaocc