fix: Worker 并发默认值从 1 改为 4(部署脚本) #1301

Merged
xiaoxia merged 1 commits from fix/worker-concurrency-default-4 into develop 2026-08-08 14:32:43 +08:00
Owner

问题

PR #1292 预期 Worker 并发数为 4,但部署脚本硬编码了 WORKER_CONCURRENCY=1,导致实际只有 1 个并发。

修改

三个部署脚本统一改为从环境变量读取,默认值 4:

  • infra/docker/deploy-staging-registry.sh${WORKER_CONCURRENCY:-4}
  • infra/docker/deploy-production.sh${WORKER_CONCURRENCY:-4}
  • infra/docker/deploy-production-registry.sh${WORKER_CONCURRENCY:-4}

验证

仅修改部署配置,无业务代码变更。

## 问题 PR #1292 预期 Worker 并发数为 4,但部署脚本硬编码了 `WORKER_CONCURRENCY=1`,导致实际只有 1 个并发。 ## 修改 三个部署脚本统一改为从环境变量读取,默认值 4: - `infra/docker/deploy-staging-registry.sh` → `${WORKER_CONCURRENCY:-4}` - `infra/docker/deploy-production.sh` → `${WORKER_CONCURRENCY:-4}` - `infra/docker/deploy-production-registry.sh` → `${WORKER_CONCURRENCY:-4}` ## 验证 仅修改部署配置,无业务代码变更。
xiaoxia added 1 commit 2026-08-08 14:23:22 +08:00
fix: Worker 并发默认值从 1 改为 4
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 34s
CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped
CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 50s
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 56s
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m30s
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m48s
AI Code Review / AI Code Review (pull_request) Successful in 1m55s
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 2m1s
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 2m3s
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m43s
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m26s
CI/CD Pipeline / Integration Tests (pull_request) Successful in 1m17s
CI/CD Pipeline / Unit Tests (pull_request) Successful in 7m2s
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
CI/CD Pipeline / CI Gate (pull_request) Successful in 55s
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 43s
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 1m3s
563131917e
deploy-staging-registry.sh / deploy-production.sh / deploy-production-registry.sh
三处 WORKER_CONCURRENCY 统一改为从环境变量读取,默认值 4。

此前硬编码为 1,导致 staging 和 production 各只有 1 个 Worker 并发,
与 PR #1292 的预期(4 并发)不符。

🚀 预览环境已部署

项目 详情
PR号 #1301
预览链接 https://pr-1301.preview.xiaoxiajianji.com
API环境 staging

💡 预览环境使用 staging API 数据,请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后,预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1301 | | 预览链接 | [https://pr-1301.preview.xiaoxiajianji.com](https://pr-1301.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据,请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后,预览环境会自动清理。
Collaborator

代码审查结果 - PR #1301

⚠️ 问题(0个需要修改)

💡 建议(1个可选)

  1. infra/docker/deploy-production.sh, deploy-production-registry.sh:生产环境默认并发数从1提升至4,属于较大的资源变更。建议在合并前确认容器配置的 CPU/Memory 限制是否足以支撑 4 倍并发,以及下游依赖(如数据库、缓存)的连接池配置是否匹配,避免因资源争抢导致 OOM 或响应超时。

格式检查通过 | 逻辑审查通过 | ⚠️ 建议关注性能


🤖 由 AI 代码审查机器人自动生成 | 2026-08-08 06:25:18 | 模型:

## 代码审查结果 - PR #1301 ### ⚠️ 问题(0个需要修改) 无 ### 💡 建议(1个可选) 1. **infra/docker/deploy-production.sh, deploy-production-registry.sh**:生产环境默认并发数从1提升至4,属于较大的资源变更。建议在合并前确认容器配置的 CPU/Memory 限制是否足以支撑 4 倍并发,以及下游依赖(如数据库、缓存)的连接池配置是否匹配,避免因资源争抢导致 OOM 或响应超时。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-08-08 06:25:18 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
xiaoxia merged commit 88e9311fb0 into develop 2026-08-08 14:32:43 +08:00

🗑️ 预览环境已清理

PR #1301 已关闭或合并,对应的预览环境已被清理。

如有需要,可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1301 已关闭或合并,对应的预览环境已被清理。 > 如有需要,可以重新打开 PR 来重新生成预览环境。
Sign in to join this conversation.