fix issue: #16, tag ordering (#17)

* fix issue: #16, tag ordering

* Refactor: tidy up

Co-authored-by: Hidetake Iwata <int128@gmail.com>

Author: nelg

main.tf

@@ -124,7 +124,22 @@ resource "aws_autoscaling_group" "this" {
     }
   }
 
-  tags = local.asg_tags
+  // Generate asg tags from default tag list
+  dynamic "tag" {
+    for_each = var.tags
+    content {
+      key                 = tag.key
+      value               = tag.value
+      propagate_at_launch = true
+    }
+  }
+
+  // Tag for name
+  tag {
+    key                 = "Name"
+    value               = "nat-instance-${var.name}"
+    propagate_at_launch = true
+  }
 
   lifecycle {
     create_before_destroy = true

variables.tf

@@ -72,20 +72,4 @@ locals {
     var.tags, {
       Name = "nat-instance-${var.name}"
   })
-
-  // Generate asg tags by merging variables in object format
-  asg_tags = concat([
-    for key, value in var.tags : {
-      key                 = key
-      value               = value
-      propagate_at_launch = true
-    }
-    ], [
-    {
-      key                 = "Name"
-      value               = "nat-instance-${var.name}"
-      propagate_at_launch = true
-    }
-    ]
-  )
 }