Merge pull request #116 from maddevsio/Feature/add_depends_on_eks_modify_readme

Feature/add depends on eks modify readme

Author: and-babkin

README.md

@@ -80,6 +80,7 @@ You can find more about this project in Anton Babenko stream:
       - [Base project name](#base-project-name)
       - [Unique prefix of resource names](#unique-prefix-of-resource-names)
       - [Separators](#separators)
+      - [Depends_on](#depends_on)
       - [Resource names](#resource-names)
       - [Variable names](#variable-names)
       - [Output names](#output-names)
@@ -570,6 +571,19 @@ The `local.name` value is then used as a prefix for all `name` and `name_prefix`
 
 > Use `name_prefix` where possible
 
+#### Depends_on
+
+When you need to add `depends_on` to a resource or a module you should put it at the end of the block with empty line in front of it.
+
+```
+resource "aws_eks_addon" "coredns" {
+  ...
+  addon_version     = var.addon_coredns_version
+
+  depends_on = [module.eks]
+}
+```
+
 #### Resource names
 
 - The resource type should not be duplicated in the resource name (either partially or in full):

terraform/layer1-aws/aws-eks.tf

@@ -178,6 +178,7 @@ EOT
       })
     }
   }
+
   depends_on = [module.vpc]
 }
 
@@ -192,6 +193,8 @@ resource "aws_eks_addon" "vpc_cni" {
   tags = {
     Environment = local.env
   }
+
+  depends_on = [module.eks]
 }
 
 resource "aws_eks_addon" "kube_proxy" {
@@ -205,6 +208,8 @@ resource "aws_eks_addon" "kube_proxy" {
   tags = {
     Environment = local.env
   }
+
+  depends_on = [module.eks]
 }
 
 resource "aws_eks_addon" "coredns" {
@@ -214,8 +219,10 @@ resource "aws_eks_addon" "coredns" {
   addon_name        = "coredns"
   resolve_conflicts = "OVERWRITE"
   addon_version     = var.addon_coredns_version
-  depends_on        = [module.eks]
+
   tags = {
     Environment = local.env
   }
+
+  depends_on = [module.eks]
 }