xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 16 Pull Requests 11 Actions 34.2k Packages Projects Releases 7 Wiki Activity

fix: add redis_url field to WorkerSettings to pick up REDIS_URL env var
CI/CD Pipeline / Production Browser E2E (pull_request) Failing after 1560h43m4s
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Failing after 1560h43m6s
Details
CI/CD Pipeline / Build Production Runtime Images (pull_request) Failing after 1560h43m8s
Details
CI/CD Pipeline / Deploy Production (pull_request) Failing after 1560h43m6s
Details
CI/CD Pipeline / Build & Push Staging (Watchtower auto-deploy) (pull_request) Failing after 1560h43m8s
Details
CI/CD Pipeline / Validate Code Quality And Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Failing after 1561h14m41s
Details

Browse Source 
Root cause of flag not working in staging:
- WorkerSettings had no 'redis_url' field, so REDIS_URL env var was ignored
- RenderEngineResolver fallback to broker_url (default redis://redis:6379/0)
- Staging redis hostname is xiaoxia-redis-staging, so connection failed
- Resolver degraded to in-memory store, flag always defaulted to disabled (legacy)

Fix: add redis_url field so REDIS_URL env var is properly mapped
This commit is contained in:
CI Bot
2026-07-13 14:44:31 +08:00
parent c48ddeef7d
commit 2fbdb8f992
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/worker/worker_app/core/config.py
+1
View File
@@ -17,6 +17,7 @@ class WorkerSettings(BaseSettings):
database_pool_recycle: int = 3600
environment: str = "development"
auto_create_schema: bool = False
redis_url: str = "redis://redis:6379/0"
# 渲染引擎选择：legacy=旧VideoComposeService，unified=新UnifiedRenderService
render_engine: str = "legacy"