Skip to content

fix(hirag): use container DNS for Supabase realtime #335

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
POWERFULMOVES merged 1 commit into from
Dec 19, 2025
Merged

fix(hirag): use container DNS for Supabase realtime #335

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pmoves/docker-compose.gpu-image.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ services:
- NEO4J_USER=${NEO4J_USER}
- NEO4J_PASSWORD=${NEO4J_PASSWORD}
- SUPA_REST_URL=${SUPA_REST_URL:-http://host.docker.internal:65421/rest/v1}
- SUPABASE_REALTIME_URL=${SUPABASE_REALTIME_URL:-ws://host.docker.internal:65421/realtime/v1}
- SUPABASE_REALTIME_URL=${SUPABASE_REALTIME_URL:-ws://supabase_kong_PMOVES.AI:8000/realtime/v1}
- OLLAMA_URL=${OLLAMA_URL:-http://pmoves-ollama:11434}
- OLLAMA_EMBED_MODEL=${OLLAMA_EMBED_MODEL:-embeddinggemma:300m}
- TENSORZERO_BASE_URL=${TENSORZERO_BASE_URL:-http://tensorzero-gateway:3000}
Expand Down
4 changes: 2 additions & 2 deletions pmoves/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -569,7 +569,7 @@ services:
- TAILSCALE_ONLY=${TAILSCALE_ONLY:-false}
- TAILSCALE_CIDRS=${TAILSCALE_CIDRS:-100.64.0.0/10}
- SUPA_REST_URL=${SUPA_REST_URL:-http://host.docker.internal:65421/rest/v1}
- SUPABASE_REALTIME_URL=${SUPABASE_REALTIME_URL:-ws://host.docker.internal:65421/realtime/v1}
- SUPABASE_REALTIME_URL=${SUPABASE_REALTIME_URL:-ws://supabase_kong_PMOVES.AI:8000/realtime/v1}
# In single-env mode, credentials come from env_file; avoid overriding with empty host vars.
extra_hosts:
- "host.docker.internal:host-gateway"
Expand Down Expand Up @@ -674,7 +674,7 @@ services:
- PGPASSWORD=${POSTGRES_PASSWORD:-pmoves}
- PGDATABASE=${POSTGRES_DB:-pmoves}
- SUPA_REST_URL=${SUPA_REST_URL:-http://host.docker.internal:65421/rest/v1}
- SUPABASE_REALTIME_URL=${SUPABASE_REALTIME_URL:-ws://host.docker.internal:65421/realtime/v1}
- SUPABASE_REALTIME_URL=${SUPABASE_REALTIME_URL:-ws://supabase_kong_PMOVES.AI:8000/realtime/v1}
# In single-env mode, credentials come from env_file; avoid overriding with empty host vars.
- OLLAMA_URL=${OLLAMA_URL:-http://pmoves-ollama:11434}
- OLLAMA_EMBED_MODEL=${OLLAMA_EMBED_MODEL:-qwen3-embedding:4b}
Expand Down
2 changes: 1 addition & 1 deletion pmoves/services/hi-rag-gateway-v2/scripts/wait-for-deps.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@

set -e

SUPABASE_URL="${SUPABASE_REALTIME_URL:-ws://host.docker.internal:65421/realtime/v1}"
SUPABASE_URL="${SUPABASE_REALTIME_URL:-ws://supabase_kong_PMOVES.AI:8000/realtime/v1}"
DISABLED="${SUPABASE_REALTIME_DISABLED:-false}"
Comment on lines 14 to 17

@chatgpt-codex-connector chatgpt-codex-connector Bot Dec 19, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Realtime default uses non-existent Supabase host

The new fallback SUPABASE_REALTIME_URL points at ws://supabase_kong_PMOVES.AI:8000/realtime/v1, but the Supabase CLI stack we join on pmoves-net exposes Kong as supabase-kong (see the botz-gateway defaults in pmoves/docker-compose.yml lines 832-839). There is no container registering the _PMOVES.AI hostname, so on a stock make -C pmoves supa-start run the wait-for-deps probe will DNS fail and Hi-RAG’s realtime listener will never come up unless users override the variable. This regresses the out-of-the-box configuration you were trying to fix for WSL.

Useful? React with 👍 / 👎.

MAX_WAIT="${WAIT_FOR_DEPS_MAX_WAIT:-120}"
INTERVAL="${WAIT_FOR_DEPS_INTERVAL:-5}"
Expand Down