Skip to content

Commit 43e867a

Browse files
author
Steven Nemetz
committed
AWS 1.6 has bugs with NLB. Test with 1.5. Hardcode required settings.
1 parent e754c47 commit 43e867a

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

main.tf

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@
33
# and S3 for logging, retrieve SSL cert from ACM
44
# ?? security groups, dns
55
#
6-
# AWS provider 1.6 had some breaking changes. This supports 1.6
6+
# AWS provider 1.6 has bugs that prevent NLBs management. Issue #2708
7+
# Currently supporting 1.5.0
78
#
89
# https://www.terraform.io/docs/providers/aws/r/lb.html
910
# https://www.terraform.io/docs/providers/aws/r/lb_listener.html
@@ -269,11 +270,14 @@ resource "aws_lb_target_group" "network" {
269270
#deregistration_delay = "${}"
270271
#target_type = "${}"
271272
health_check {
272-
interval = "${var.health_check_interval}"
273+
interval = "10" # only 10, 30 valid. Cannot be changed after creation
273274
port = "${var.health_check_port}"
274275
healthy_threshold = "${var.health_check_healthy_threshold}"
275276
unhealthy_threshold = "${var.health_check_unhealthy_threshold}"
276277
protocol = "${var.health_check_protocol}"
278+
timeout = "6"
279+
path = "/"
280+
matcher = "200-399"
277281
}
278282
tags = "${module.label.tags}"
279283
}

test/providers.tf

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11

22
provider "aws" {
33
region = "${var.region}"
4-
#version = "1.5"
4+
version = "1.5"
5+
# Can not upgrade until bugs introduced in 1.6 are fixed
56
}

0 commit comments

Comments
 (0)