Skip to content

Commit

Permalink
fix: add missing envoy app port mapping
Browse files Browse the repository at this point in the history
  • Loading branch information
saefty committed Mar 6, 2025
1 parent 52d0bc5 commit 788e61c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,9 @@ resource "aws_ecs_task_definition" "this" {
type = "APPMESH"

properties = {
AppPorts = var.container_port
AppPorts = join(",", concat([tostring(var.container_port)], [
for x in var.extra_port_mappings : x.containerPort
]))
EgressIgnoredIPs = "169.254.170.2,169.254.169.254"
IgnoredGID = "1337"
ProxyEgressPort = 15001
Expand Down
2 changes: 1 addition & 1 deletion variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -327,7 +327,7 @@ variable "efs_volumes" {
variable "extra_port_mappings" {
default = []
description = "Additional ports to be exposed from the container."
type = list(object({
type = list(object({
hostPort = number
containerPort = number
protocol = optional(string, "tcp")
Expand Down

0 comments on commit 788e61c

Please sign in to comment.