fix: refactor into common uri parsing logic, fix test, and fix compose file (#5261)
refactor into common uri parsing logic, fix test, and fix compose file
This commit is contained in:
@@ -11,13 +11,11 @@ jobs:
|
||||
runs-on: ubuntu-latest
|
||||
timeout-minutes: 15
|
||||
env:
|
||||
# Database configuration for docker-compose
|
||||
# Note: LETTA_PG_HOST is intentionally NOT set here because:
|
||||
# - Inside containers: uses 'letta_db' (from dev-compose.yaml defaults)
|
||||
# - Host pytest tests: uses 'localhost' (to connect via port mapping)
|
||||
# Database configuration - these will be used by dev-compose.yaml
|
||||
LETTA_PG_DB: letta
|
||||
LETTA_PG_USER: letta
|
||||
LETTA_PG_PASSWORD: letta
|
||||
LETTA_PG_HOST: pgvector_db # Internal Docker service name
|
||||
LETTA_PG_PORT: 5432
|
||||
# Server configuration for tests
|
||||
LETTA_SERVER_PASS: test_server_token
|
||||
|
||||
@@ -31,9 +31,9 @@ services:
|
||||
- LETTA_PG_DB=${LETTA_PG_DB:-letta}
|
||||
- LETTA_PG_USER=${LETTA_PG_USER:-letta}
|
||||
- LETTA_PG_PASSWORD=${LETTA_PG_PASSWORD:-letta}
|
||||
- LETTA_PG_HOST=${LETTA_PG_HOST:-letta_db}
|
||||
- LETTA_PG_HOST=${LETTA_PG_HOST:-pgvector_db}
|
||||
- LETTA_PG_PORT=${LETTA_PG_PORT:-5432}
|
||||
- LETTA_PG_URI=postgresql://${LETTA_PG_USER:-letta}:${LETTA_PG_PASSWORD:-letta}@${LETTA_PG_HOST:-letta_db}:${LETTA_PG_PORT:-5432}/${LETTA_PG_DB:-letta}
|
||||
- LETTA_PG_URI=${LETTA_PG_URI:-postgresql://${LETTA_PG_USER:-letta}:${LETTA_PG_PASSWORD:-letta}@${LETTA_PG_HOST:-pgvector_db}:${LETTA_PG_PORT:-5432}/${LETTA_PG_DB:-letta}}
|
||||
- LETTA_DEBUG=True
|
||||
- OPENAI_API_KEY=${OPENAI_API_KEY}
|
||||
- GROQ_API_KEY=${GROQ_API_KEY}
|
||||
|
||||
Reference in New Issue
Block a user