From 42798ff0559e786f993e97d94481e24860ebd71a Mon Sep 17 00:00:00 2001 From: Manish Kumar Date: Tue, 11 Aug 2026 21:16:14 -0500 Subject: [PATCH] fix(#196): add GATEWAY_URL to celery-worker workbench already had this; celery-worker didn't. The 4 TES call sites fixed in omnibioai-workbench#196 (bio_agent's two factories, step_executor.py, workflow_runner/views.py, catalog/services.py) now read GATEWAY_URL exclusively to route through api-gateway instead of straight to TES. Deliberately NOT repointing TES_BASE_URL/OMNIBIOAI_TES_URL to the gateway on either service: traced every remaining reader of those two vars first, and found two more (bioquery_tasks.py's Celery flow, filed as omnibioai-workbench#209; and a separate OMNIBIOAI_TES_PROFILES routing subsystem in core/config.py, filed as #210) that carry no bearer token at all. Repointing the shared env vars would have silently 401'd those instead of leaving them at their pre-existing (bypassing, but working) behavior. Co-Authored-By: Claude Sonnet 5 --- docker-compose.yml | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/docker-compose.yml b/docker-compose.yml index a9449c6..de40954 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -510,6 +510,16 @@ services: CELERY_BROKER_URL: redis://redis:6379/1 CELERY_RESULT_BACKEND: redis://redis:6379/2 TES_BASE_URL: http://tes:8081 + # #196: the 4 fixed TES call sites (bio_agent's factories, + # step_executor.py, workflow_runner/views.py, catalog/services.py) + # now read GATEWAY_URL exclusively, not TES_BASE_URL/ + # OMNIBIOAI_TES_URL -- workbench already had this var, celery-worker + # didn't. TES_BASE_URL above is deliberately left pointed directly at + # TES (not repointed to the gateway) -- other, not-yet-fixed readers + # of it in this same image (bioquery_tasks.py, #209) have no bearer + # token attached and would silently 401 if this var suddenly routed + # through gateway's AuthMiddleware. + GATEWAY_URL: http://api-gateway:8080 TOOLSERVER_BASE_URL: http://toolserver:9090 MODEL_REGISTRY_BASE_URL: http://model-registry:8095 ORCHESTRATOR_MAX_RETRIES: "10"