From 4f60ef453f7c9179a21739ac25f440d2b9e7b4ae Mon Sep 17 00:00:00 2001
From: Viktor Barzin <viktorbarzin@meta.com>
Date: Wed, 25 Feb 2026 23:46:47 +0000
Subject: [PATCH] fix: hardcode nginx-k8s.conf in dashboard Dockerfile

The buildx plugin was not passing NGINX_CONF build arg correctly,
causing the docker-compose nginx config (with hostname api-gateway)
to be used instead of the K8s one (with localhost).
---
 .woodpecker.yml             | 2 --
 docker/Dockerfile.dashboard | 5 ++---
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/.woodpecker.yml b/.woodpecker.yml
index 30f483f..65f9c43 100644
--- a/.woodpecker.yml
+++ b/.woodpecker.yml
@@ -48,8 +48,6 @@ steps:
       repo: viktorbarzin/trading-bot-dashboard
       dockerfile: docker/Dockerfile.dashboard
       context: .
-      build_args:
-        - NGINX_CONF=docker/nginx-k8s.conf
       cache_from: viktorbarzin/trading-bot-dashboard:latest
       tags:
         - "${CI_PIPELINE_NUMBER}"
diff --git a/docker/Dockerfile.dashboard b/docker/Dockerfile.dashboard
index e0deb45..2f8ebd3 100644
--- a/docker/Dockerfile.dashboard
+++ b/docker/Dockerfile.dashboard
@@ -23,9 +23,8 @@ FROM nginx:alpine
 # Remove default nginx site
 RUN rm /etc/nginx/conf.d/default.conf
 
-# Copy custom nginx config (override NGINX_CONF for K8s builds)
-ARG NGINX_CONF=docker/nginx.conf
-COPY ${NGINX_CONF} /etc/nginx/conf.d/default.conf
+# Copy K8s nginx config (api-gateway on localhost since they share a pod)
+COPY docker/nginx-k8s.conf /etc/nginx/conf.d/default.conf
 
 # Copy built assets from the builder stage
 COPY --from=builder /app/dist /usr/share/nginx/html