Skip to content

Commit 0f43130

Browse files
committed
Switch to using boolean module for enabled
1 parent 8d807d9 commit 0f43130

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

main.tf

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,12 @@
44
# https://www.terraform.io/docs/providers/aws/r/efs_file_system.html
55
# https://www.terraform.io/docs/providers/aws/r/efs_mount_target.html
66

7+
module "enabled" {
8+
source = "devops-workflow/boolean/local"
9+
version = "0.1.0"
10+
value = "${var.enabled}"
11+
}
12+
713
# Define composite variables for resources
814
module "label" {
915
source = "devops-workflow/label/local"
@@ -19,22 +25,22 @@ module "label" {
1925
}
2026

2127
resource "aws_efs_file_system" "default" {
22-
count = "${var.enabled ? 1 : 0}"
28+
count = "${module.enabled.value}"
2329
performance_mode = "${var.performance_mode}"
2430
encrypted = "${var.encrypted}"
2531
kms_key_id = "${var.kms_key_id}"
2632
tags = "${module.label.tags}"
2733
}
2834

2935
resource "aws_efs_mount_target" "default" {
30-
count = "${var.enabled ? length(compact(var.subnets)) : 0}"
36+
count = "${module.enabled.value ? length(compact(var.subnets)) : 0}"
3137
file_system_id = "${aws_efs_file_system.default.id}"
3238
subnet_id = "${element(compact(var.subnets), count.index)}"
3339
security_groups = ["${aws_security_group.default.id}"]
3440
}
3541

3642
resource "aws_security_group" "default" {
37-
count = "${var.enabled ? 1 : 0}"
43+
count = "${module.enabled.value}"
3844
name = "${module.label.id}"
3945
description = "EFS Access"
4046
vpc_id = "${var.vpc_id}"
@@ -45,7 +51,7 @@ resource "aws_security_group" "default" {
4551
}
4652

4753
resource "aws_security_group_rule" "ingress" {
48-
count = "${var.enabled ? length(compact(var.security_groups)) : 0}"
54+
count = "${module.enabled.value ? length(compact(var.security_groups)) : 0}"
4955
type = "ingress"
5056
from_port = "2049"
5157
to_port = "2049"
@@ -55,7 +61,7 @@ resource "aws_security_group_rule" "ingress" {
5561
}
5662

5763
resource "aws_security_group_rule" "egress" {
58-
count = "${var.enabled ? 1 : 0}"
64+
count = "${module.enabled.value}"
5965
type = "egress"
6066
from_port = 0
6167
to_port = 0
@@ -72,5 +78,5 @@ module "dns" {
7278
ttl = "${var.dns_ttl}"
7379
zone_id = "${var.zone_id}"
7480
records = ["${element(concat(aws_efs_file_system.default.*.dns_name, list("")),0)}"]
75-
enabled = "${var.enabled ? (length(var.zone_id) > 0 ? "true" : "false") : "false"}"
81+
enabled = "${module.enabled.value ? (length(var.zone_id) > 0 ? "true" : "false") : "false"}"
7682
}

0 commit comments

Comments
 (0)