fix(docker): Remove environment variable overrides in docker-compose.yml (#1411)
The docker-compose.yml had an `environment:` section with variable
substitutions (${VAR:-}) that was overriding values from .llm.env with
empty strings.
- Commented out the `environment:` section to prevent overwrites
- Added clear warning comment explaining the override behavior
- .llm.env values now load directly into container without interference
This commit is contained in:
@@ -6,15 +6,16 @@ x-base-config: &base-config
|
|||||||
- "11235:11235" # Gunicorn port
|
- "11235:11235" # Gunicorn port
|
||||||
env_file:
|
env_file:
|
||||||
- .llm.env # API keys (create from .llm.env.example)
|
- .llm.env # API keys (create from .llm.env.example)
|
||||||
environment:
|
# Uncomment to set default environment variables (will overwrite .llm.env)
|
||||||
- OPENAI_API_KEY=${OPENAI_API_KEY:-}
|
# environment:
|
||||||
- DEEPSEEK_API_KEY=${DEEPSEEK_API_KEY:-}
|
# - OPENAI_API_KEY=${OPENAI_API_KEY:-}
|
||||||
- ANTHROPIC_API_KEY=${ANTHROPIC_API_KEY:-}
|
# - DEEPSEEK_API_KEY=${DEEPSEEK_API_KEY:-}
|
||||||
- GROQ_API_KEY=${GROQ_API_KEY:-}
|
# - ANTHROPIC_API_KEY=${ANTHROPIC_API_KEY:-}
|
||||||
- TOGETHER_API_KEY=${TOGETHER_API_KEY:-}
|
# - GROQ_API_KEY=${GROQ_API_KEY:-}
|
||||||
- MISTRAL_API_KEY=${MISTRAL_API_KEY:-}
|
# - TOGETHER_API_KEY=${TOGETHER_API_KEY:-}
|
||||||
- GEMINI_API_TOKEN=${GEMINI_API_TOKEN:-}
|
# - MISTRAL_API_KEY=${MISTRAL_API_KEY:-}
|
||||||
- LLM_PROVIDER=${LLM_PROVIDER:-} # Optional: Override default provider (e.g., "anthropic/claude-3-opus")
|
# - GEMINI_API_KEY=${GEMINI_API_KEY:-}
|
||||||
|
# - LLM_PROVIDER=${LLM_PROVIDER:-} # Optional: Override default provider (e.g., "anthropic/claude-3-opus")
|
||||||
volumes:
|
volumes:
|
||||||
- /dev/shm:/dev/shm # Chromium performance
|
- /dev/shm:/dev/shm # Chromium performance
|
||||||
deploy:
|
deploy:
|
||||||
|
|||||||
Reference in New Issue
Block a user