Skip to content

Commit

Permalink
fix(aws-ecs): no longer need port override (#28)
Browse files Browse the repository at this point in the history
  • Loading branch information
jordan-acosta committed Feb 28, 2024
1 parent ed4e180 commit 8adca92
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion create-an-aws-ecs-byoc-app/components/ecs-service/alb.tf
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ module "ingress" {
api = {
name_prefix = "api-"
protocol = "HTTP"
port = 5000
port = 8080
target_type = "ip"
create_attachment = false
health_check = {
Expand Down
6 changes: 3 additions & 3 deletions create-an-aws-ecs-byoc-app/components/ecs-service/service.tf
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@ module "service" {
port_mappings = [
{
name = "http"
containerPort = 5000
hostPort = 5000
containerPort = 8080
hostPort = 8080
protocol = "http"
}
]
Expand All @@ -43,7 +43,7 @@ module "service" {
service = {
target_group_arn = module.ingress.target_groups["api"].arn
container_name = "api"
container_port = 5000
container_port = 8080
}
}

Expand Down

0 comments on commit 8adca92

Please sign in to comment.