From 811688bd0e106238124b925cc02693b959208980 Mon Sep 17 00:00:00 2001
From: Matthew Creal Morgan <matthew.c.morgan@census.gov>
Date: Tue, 17 Sep 2024 05:35:39 -0700
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B=20fix(dns=5Fzones.tf):=20removed?=
 =?UTF-8?q?=20unneeded=20tag=20filter=20on=20dummy-vpc=20(#23)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Hit an error running Terragrunt, too many vpcs in the environment. Turns
out each of the clusters was creating it's own dummy vpc and tagging it
as for that cluster since we had a cluster specific tag filter
---
 dns_zones.tf | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/dns_zones.tf b/dns_zones.tf
index a5d3dac..0c5b1fd 100644
--- a/dns_zones.tf
+++ b/dns_zones.tf
@@ -44,11 +44,10 @@ provider "aws" {
 
 provider "aws" {
   alias = "self"
-  # Commented as in testing we are assuming this role already
-  # assume_role {
-  #   role_arn     = format("arn:%v:iam::%v:role/r-inf-terraform", data.aws_arn.current.partition, data.aws_caller_identity.current.account_id)
-  #   session_name = var.os_username
-  # }
+  assume_role {
+    role_arn     = format("arn:%v:iam::%v:role/r-inf-terraform-route53", data.aws_arn.current.partition, data.aws_caller_identity.current.account_id)
+    session_name = var.os_username
+  }
 }
 
 #-------------------------------------------------
@@ -125,10 +124,6 @@ data "aws_vpc" "dummy_vpc" {
     name   = "tag:Name"
     values = ["vpc0-dummy"]
   }
-  filter {
-    name   = "tag:eks-cluster-name"
-    values = [var.cluster_name]
-  }
 }
 
 resource "aws_vpc" "vpc" {
@@ -137,8 +132,7 @@ resource "aws_vpc" "vpc" {
   enable_dns_hostnames = false
   tags = merge(
     var.tags,
-    { "Name" = "vpc0-dummy" },
-    { "eks-cluster-name" = var.cluster_name }
+    { "Name" = "vpc0-dummy" }
   )
 }