Merge pull request #4 from Janus-vistaprint/fix_destroy_timing

removing create before destroy to fix rebuild timing
Janus-vistaprint · May 8, 2017 · 6e347d8 · 6e347d8
2 parents 0b88171 + c49797b
commit 6e347d8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/tf_ecs_default_service/main.tf b/tf_ecs_default_service/main.tf
@@ -24,9 +24,11 @@ resource "aws_ecs_service" "svc" {
     field = "instanceId"
   }
 
+  /*  enabling this breaks teardown/rebuild
   lifecycle {
     create_before_destroy = true
   }
+  */
 }
 
 data "template_file" "task_definition" {

diff --git a/tf_ecs_service/main.tf b/tf_ecs_service/main.tf
@@ -24,9 +24,11 @@ resource "aws_ecs_service" "svc" {
     field = "instanceId"
   }
 
-  lifecycle {
-    create_before_destroy = true
-  }
+  # We have found that create_before_destroy breaks teardown.  
+  #lifecycle {
+  #  create_before_destroy = true
+  #}
+
 }
 
 data "template_file" "task_definition" {