From 2515e0e561509d026fd0d4725ab0bd864e7340f9 Mon Sep 17 00:00:00 2001
From: Bryant Biggs <bryantbiggs@gmail.com>
Date: Mon, 7 Feb 2022 15:33:35 -0500
Subject: [PATCH] fix: Add node group dependency for EKS addons resource
 creation (#1840)

---
 main.tf | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/main.tf b/main.tf
index e7f1f113b8..7fab72e6f9 100644
--- a/main.tf
+++ b/main.tf
@@ -234,6 +234,16 @@ resource "aws_eks_addon" "this" {
     ]
   }
 
+  # Note: if an addon needs to be provisioned ahead of a node group users will
+  # need to create the addon outside of this module until a 2nd addon resource is added
+  # to the module (here) that is not dependent on node groups
+  # Or if addon management improves, this dependency can be removed https://github.com/aws/containers-roadmap/issues/1389
+  depends_on = [
+    module.fargate_profile,
+    module.eks_managed_node_group,
+    module.self_managed_node_group,
+  ]
+
   tags = var.tags
 }