From 1c7d0ee99629d045df1b44c1282f15ddb63dfa78 Mon Sep 17 00:00:00 2001
From: maxim <nikromax@gmail.com>
Date: Thu, 4 Nov 2021 18:30:21 +0600
Subject: [PATCH] refactor: switch external-dns helm chart from bitnami helm
 repository to https://github.com/kubernetes-sigs/external-dns

---
 terraform/layer2-k8s/eks-external-dns.tf         |  1 -
 terraform/layer2-k8s/helm-charts.yaml            |  4 ++--
 terraform/layer2-k8s/templates/external-dns.yaml | 13 +++++--------
 3 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/terraform/layer2-k8s/eks-external-dns.tf b/terraform/layer2-k8s/eks-external-dns.tf
index 5792d793..3dd9656b 100644
--- a/terraform/layer2-k8s/eks-external-dns.tf
+++ b/terraform/layer2-k8s/eks-external-dns.tf
@@ -13,7 +13,6 @@ data "template_file" "external_dns" {
     role_arn    = module.aws_iam_external_dns.role_arn
     domain_name = local.domain_name
     zone_type   = "public"
-    region      = local.region
   }
 }
 
diff --git a/terraform/layer2-k8s/helm-charts.yaml b/terraform/layer2-k8s/helm-charts.yaml
index ef9f935b..bd883076 100644
--- a/terraform/layer2-k8s/helm-charts.yaml
+++ b/terraform/layer2-k8s/helm-charts.yaml
@@ -33,8 +33,8 @@ charts:
     version:
   - id: external-dns
     chart: external-dns
-    repository: https://charts.bitnami.com/bitnami
-    version: 4.9.4
+    repository: https://kubernetes-sigs.github.io/external-dns
+    version: 1.5.0
   - id: external-secrets
     chart: kubernetes-external-secrets
     repository: https://external-secrets.github.io/kubernetes-external-secrets
diff --git a/terraform/layer2-k8s/templates/external-dns.yaml b/terraform/layer2-k8s/templates/external-dns.yaml
index 4ce68356..c5e68194 100644
--- a/terraform/layer2-k8s/templates/external-dns.yaml
+++ b/terraform/layer2-k8s/templates/external-dns.yaml
@@ -9,15 +9,12 @@ serviceAccount:
 
 provider: aws
 domainFilters: [${domain_name}]
-aws:
-  region: ${region}
-  zoneType: ${zone_type}
+extraArgs:
+  - --aws-zone-type=${zone_type}
+  - --aws-batch-change-size=1000
 
-metrics:
-  enabled: true
-  serviceMonitor:
-    enabled: false
-    namespace: monitoring
+serviceMonitor:
+  enabled: false
 
 sources:
   - service