Add storage service URLs to env.example for cross-environment deployment#25
Add storage service URLs to env.example for cross-environment deployment#25
Conversation
…eployment Add REDIS_URI, QDRANT_URL, and MEMGRAPH_URI to env.example so users can configure different storage addresses for dev/test/prod environments. Co-authored-by: BukeLy <19304666+BukeLy@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR documents three previously undocumented storage service connection URL environment variables (REDIS_URI, QDRANT_URL, MEMGRAPH_URI) in the env.example file. These variables were already supported in src/config.py but missing from the example configuration, causing users to incorrectly assume that storage service addresses were hardcoded to Docker-internal hostnames. This documentation enables users to properly configure external storage services for cross-environment deployments (development, testing, production).
- Adds documentation for storage service connection URLs under the "存储后端配置" section
- Includes clear warnings about when these variables need to be configured
- Provides examples for both Docker-internal (default) and external deployments
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot implement #25 (review) |
Review #3577353134 is an auto-generated PR overview summary, not a request for changes. It confirms the PR correctly:
The PR is complete as-is. If you have specific changes you'd like me to make, please let me know. |
REDIS_URI,QDRANT_URL, andMEMGRAPH_URIare supported insrc/config.pybut undocumented inenv.example, causing users to believe these are hardcoded Docker-internal addresses.Changes
env.exampleunder "存储后端配置" section:REDIS_URI- Redis/DragonflyDB connectionQDRANT_URL- Qdrant vector DB connectionMEMGRAPH_URI- Memgraph graph DB connectionUsage
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.