Adding an option to apply tags to the launch configuration security g…

…roup
EFXCIA · Aug 16, 2018 · c74baad · c74baad
1 parent c92c6cb
commit c74baad
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/modules/consul-cluster/main.tf b/modules/consul-cluster/main.tf
@@ -39,7 +39,7 @@ resource "aws_autoscaling_group" "autoscaling_group" {
       value               = "${var.cluster_tag_value}"
       propagate_at_launch = true
     },
-    "${var.tags}",
+    "${var.tags}"
   ]
 }
 
@@ -96,9 +96,7 @@ resource "aws_security_group" "lc_security_group" {
     create_before_destroy = true
   }
 
-  tags {
-    Name = "${var.cluster_name}"
-  }
+  tags = "${merge(map("Name", var.cluster_name), var.security_group_tags)}"
 }
 
 resource "aws_security_group_rule" "allow_ssh_inbound" {

diff --git a/modules/consul-cluster/variables.tf b/modules/consul-cluster/variables.tf
@@ -83,6 +83,12 @@ variable "allowed_inbound_security_group_ids" {
   default     = []
 }
 
+variable "security_group_tags" {
+  description = "Tags to be applied to the LC security group"
+  type        = "map"
+  default     = {}
+}
+
 variable "termination_policies" {
   description = "A list of policies to decide how the instances in the auto scale group should be terminated. The allowed values are OldestInstance, NewestInstance, OldestLaunchConfiguration, ClosestToNextInstanceHour, Default."
   default     = "Default"