Skip to content

Commit 5976c5a

Browse files
committed
Minor fixes
1 parent 806e47c commit 5976c5a

File tree

1 file changed

+1
-6
lines changed
  • operations/deployment/terraform/modules/aws/ecs

1 file changed

+1
-6
lines changed

operations/deployment/terraform/modules/aws/ecs/aws_ecs.tf

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ resource "aws_ecs_task_definition" "ecs_task" {
4040
"image": local.aws_ecs_app_image[count.index],
4141
"cpu": local.aws_ecs_container_cpu[count.index],
4242
"memory": local.aws_ecs_container_mem[count.index],
43-
"networkMode": "awsvpc",
4443
"portMappings": [
4544
{
4645
"containerPort": tonumber(local.aws_ecs_container_port[count.index]),
@@ -49,8 +48,6 @@ resource "aws_ecs_task_definition" "ecs_task" {
4948
}
5049
],
5150
"environment": local.env_repo_vars,
52-
53-
# === Log configuration ===
5451
"logConfiguration": (
5552
var.aws_ecs_cloudwatch_enable && var.aws_ecs_cloudwatch_log_driver == "awslogs" ?
5653
{
@@ -72,12 +69,10 @@ resource "aws_ecs_task_definition" "ecs_task" {
7269
} : null
7370
)
7471
},
75-
76-
# === FireLens sidecar ===
7772
var.aws_ecs_cloudwatch_enable && var.aws_ecs_cloudwatch_log_driver == "awsfirelens" ? {
7873
"name": "log_router",
7974
"image": "public.ecr.aws/aws-observability/aws-for-fluent-bit:latest",
80-
"essential": true,
75+
"essential": false,
8176
"firelensConfiguration": {
8277
"type": "fluentbit"
8378
}

0 commit comments

Comments
 (0)