Skip to content

Commit d408880

Browse files
committed
Fixing error with subnets
1 parent 0476c43 commit d408880

File tree

3 files changed

+12
-6
lines changed

3 files changed

+12
-6
lines changed

README.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ Check valid versions on:
1717

1818
module "ecs-fargate-service" {
1919
source = "cn-terraform/ecs-fargate-service/aws"
20-
version = "1.0.5"
20+
version = "1.0.6"
2121
name_preffix = var.name_preffix
2222
profile = var.profile
2323
region = var.region
@@ -26,7 +26,8 @@ Check valid versions on:
2626
container_port = module.td.container_port
2727
ecs_cluster_name = module.ecs-cluster.aws_ecs_cluster_cluster_name
2828
ecs_cluster_arn = module.ecs-cluster.aws_ecs_cluster_cluster_arn
29-
subnets = module.networking.private_subnets_ids
29+
private_subnets = module.networking.private_subnets_ids
30+
public_subnets = module.networking.public_subnets_ids
3031
}
3132

3233
Check the section "Other modules that you may need to use this module" for details about modules mentioned in the usage example.

main.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ resource "aws_ecs_service" "service" {
8080
}
8181
network_configuration {
8282
security_groups =[aws_security_group.ecs_tasks_sg.id]
83-
subnets = var.subnets
83+
subnets = var.private_subnets
8484
assign_public_ip = var.assign_public_ip
8585
}
8686
load_balancer {
@@ -227,7 +227,7 @@ resource "aws_lb" "lb" {
227227
name = "${var.name_preffix}-lb"
228228
internal = false
229229
load_balancer_type = "application"
230-
subnets = var.subnets
230+
subnets = var.public_subnets
231231
security_groups = [aws_security_group.lb_sg.id]
232232
enable_deletion_protection = false
233233
enable_cross_zone_load_balancing = true

variables.tf

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,13 @@ variable "ecs_cluster_arn" {
3838
description = "ARN of an ECS cluster"
3939
}
4040

41-
variable "subnets" {
42-
description = "The subnets associated with the task or service."
41+
variable "private_subnets" {
42+
description = "The private subnets associated with the task or service."
43+
type = list
44+
}
45+
46+
variable "public_subnets" {
47+
description = "The public subnets associated with the task or service."
4348
type = list
4449
}
4550

0 commit comments

Comments
 (0)