Skip to content

Commit f102620

Browse files
committed
rename parameter alb_only
1 parent 9eaf7a8 commit f102620

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ module "ecs_apps" {
2828

2929
| Name | Description | Type | Default | Required |
3030
|------|-------------|:----:|:-----:|:-----:|
31-
| alb-only | Whether to deploy only an alb and no cloudFront or not with the cluster | string | `"false"` | no |
31+
| alb_only | Whether to deploy only an alb and no cloudFront or not with the cluster | string | `"false"` | no |
3232
| alb | Whether to deploy an ALB or not with the cluster | string | `"true"` | no |
3333
| asg\_max | Max number of instances for autoscaling group | string | `"4"` | no |
3434
| asg\_memory\_target | Target average memory percentage to track for autoscaling | string | `"60"` | no |

_variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ variable "alb" {
5959
default = true
6060
description = "Whether to deploy an ALB or not with the cluster"
6161
}
62-
variable "alb-only" {
62+
variable "alb_only" {
6363
default = false
6464
description = "Whether to deploy only an alb and no cloudFront or not with the cluster"
6565
}

waf.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
resource "aws_wafregional_web_acl_association" "alb" {
2-
count = "${var.alb && !var.alb-only ? 1 : 0}"
2+
count = "${var.alb && !var.alb_only ? 1 : 0}"
33

44
resource_arn = aws_lb.ecs[0].arn
55
web_acl_id = aws_wafregional_web_acl.alb[0].id
66
}
77

88
resource "aws_wafregional_web_acl" "alb" {
9-
count = "${var.alb && !var.alb-only ? 1 : 0}"
9+
count = "${var.alb && !var.alb_only ? 1 : 0}"
1010

1111
depends_on = ["aws_wafregional_rule.alb_header"]
1212
name = "alb_ecs_${var.name}"
@@ -28,7 +28,7 @@ resource "aws_wafregional_web_acl" "alb" {
2828
}
2929

3030
resource "aws_wafregional_rule" "alb_header" {
31-
count = "${var.alb && !var.alb-only ? 1 : 0}"
31+
count = "${var.alb && !var.alb_only ? 1 : 0}"
3232

3333
depends_on = ["aws_wafregional_byte_match_set.alb_header"]
3434
name = "alb_cloudfront_header_ecs_${var.name}"
@@ -47,7 +47,7 @@ resource "random_string" "alb_cloudfront_key" {
4747
}
4848

4949
resource "aws_wafregional_byte_match_set" "alb_header" {
50-
count = "${var.alb && !var.alb-only ? 1 : 0}"
50+
count = "${var.alb && !var.alb_only ? 1 : 0}"
5151

5252
name = "alb_cloudfront_header_ecs_${var.name}"
5353

0 commit comments

Comments
 (0)