From 4ab11e9322e6e32fc9feab7e7f22a5d76adc44ea Mon Sep 17 00:00:00 2001 From: Jingsi Zhang <17371143+tyrahappy@users.noreply.github.com> Date: Sat, 28 Mar 2026 13:00:22 -0700 Subject: [PATCH] fix: add FLAIR2_ prefix to all ECS environment variable names MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit app/config.py uses env_prefix="FLAIR2_", so all env vars injected by ECS must carry the prefix to be picked up by pydantic-settings: REDIS_URL → FLAIR2_REDIS_URL S3_BUCKET → FLAIR2_S3_BUCKET ENV → FLAIR2_ENVIRONMENT Without this fix the containers would start but settings.redis_url, settings.s3_bucket, and settings.environment would all fall back to their defaults (empty string / "dev"), silently misconfiguring the app. Applies to both API and Celery worker task definitions. Co-Authored-By: Claude Sonnet 4.6 --- terraform/modules/ecs/main.tf | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/terraform/modules/ecs/main.tf b/terraform/modules/ecs/main.tf index fab5109..cdda604 100644 --- a/terraform/modules/ecs/main.tf +++ b/terraform/modules/ecs/main.tf @@ -54,9 +54,9 @@ resource "aws_ecs_task_definition" "api" { }] environment = [ - { name = "REDIS_URL", value = var.redis_url }, - { name = "S3_BUCKET", value = var.s3_bucket_name }, - { name = "ENV", value = var.env } + { name = "FLAIR2_REDIS_URL", value = var.redis_url }, + { name = "FLAIR2_S3_BUCKET", value = var.s3_bucket_name }, + { name = "FLAIR2_ENVIRONMENT", value = var.env } ] secrets = [ @@ -144,9 +144,9 @@ resource "aws_ecs_task_definition" "worker" { ] environment = [ - { name = "REDIS_URL", value = var.redis_url }, - { name = "S3_BUCKET", value = var.s3_bucket_name }, - { name = "ENV", value = var.env } + { name = "FLAIR2_REDIS_URL", value = var.redis_url }, + { name = "FLAIR2_S3_BUCKET", value = var.s3_bucket_name }, + { name = "FLAIR2_ENVIRONMENT", value = var.env } ] secrets = [