From 2a41032414b5cef1cef573378d89392478813423 Mon Sep 17 00:00:00 2001
From: Jeremie Fraeys <jfaeys@gmail.com>
Date: Mon, 23 Feb 2026 20:30:07 -0500
Subject: [PATCH] fix(deployments): fix docker-compose build context paths

Fix build context resolution in smoke test scripts:

- docker-compose.dev.yml: Use ${FETCHML_REPO_ROOT:-..} for api-server and worker
- docker-compose.prod.smoke.yml: Simplify dockerfile path (remove redundant FETCHML_REPO_ROOT)

Previously used 'context: ..' which resolved incorrectly when docker-compose
was run with --project-directory. Now consistently uses FETCHML_REPO_ROOT env var
for proper path resolution in both dev and prod smoke tests.
---
 deployments/docker-compose.dev.yml        | 4 ++--
 deployments/docker-compose.prod.smoke.yml | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/deployments/docker-compose.dev.yml b/deployments/docker-compose.dev.yml
index b522568..9310820 100644
--- a/deployments/docker-compose.dev.yml
+++ b/deployments/docker-compose.dev.yml
@@ -33,7 +33,7 @@ services:
       retries: 3
   api-server:
     build:
-      context: ..
+      context: ${FETCHML_REPO_ROOT:-..}
       dockerfile: build/docker/simple.Dockerfile
     container_name: ml-experiments-api
     user: "0:0"
@@ -126,7 +126,7 @@ services:
     restart: "no"
   worker:
     build:
-      context: ..
+      context: ${FETCHML_REPO_ROOT:-..}
       dockerfile: build/docker/simple.Dockerfile
     container_name: ml-experiments-worker
     user: "0:0"
diff --git a/deployments/docker-compose.prod.smoke.yml b/deployments/docker-compose.prod.smoke.yml
index 84cdb07..5641c31 100644
--- a/deployments/docker-compose.prod.smoke.yml
+++ b/deployments/docker-compose.prod.smoke.yml
@@ -33,7 +33,7 @@ services:
   api-server:
     build:
       context: ${FETCHML_REPO_ROOT:-..}
-      dockerfile: ${FETCHML_REPO_ROOT:-..}/build/docker/simple.Dockerfile
+      dockerfile: build/docker/simple.Dockerfile
     user: "0:0"
     restart: unless-stopped
     expose: