fix: 部署脚本 worker health-cmd 未转义导致 staging 部署失败回滚 #1806

Merged

xiaoxia merged 1 commits from fix/deploy-script-pid-escape into develop 2026-09-08 22:41:28 +08:00

Conversation 2 Commits 1 Files Changed 1

+2 -2

xiaoxia commented 2026-09-08 22:17:14 +08:00

Owner

Copy Link

Copy Source

问题

staging 部署流水线 Deploy Staging job 失败，所有新版本镜像拉取成功后，容器启动阶段报错并自动回滚：

sh: 491: pid: parameter not set
FAIL xiaoxia-worker-staging: not created
ERROR: 1 个容器启动失败 → 自动回滚

根因

scripts/ci_staging_deploy.sh 开启了 set -eu。worker 容器的 --health-cmd 使用双引号包裹：

--health-cmd "sh -c "for pid in /proc/[0-9]*/cmdline; do if grep -ql celery "$pid" ...

其中的 $pid 被外层部署脚本在 docker run 参数展开阶段提前求值。pid 在该作用域未定义，set -u 下直接报 parameter not set 并终止，worker 容器未创建 → 部署判定失败 → 回滚到旧版本。

该 health-cmd 是 #1803 周期 Worker 健康检查修复（commit e4ac3983）引入。

修复

将 health-cmd 内 $pid 转义为 \$pid，透传到容器内由 healthcheck shell 展开：

--health-cmd "sh -c "for pid in /proc/[0-9]*/cmdline; do if grep -ql celery "\$pid" ...

• 主部署段（504 行）与回滚段（195 行）两处一并修复
• 已用 bash -n 语法检查 + 模拟 set -eu 展开验证：外层不再报错，容器内收到的 health-cmd 正确保留字面量 $pid

此修复同时解除 #1805（AI数字人 v3）合入后的 staging 部署阻塞。

🤖 Generated with Claude Code

## 问题 staging 部署流水线 Deploy Staging job 失败，所有新版本镜像拉取成功后，容器启动阶段报错并自动回滚： ``` sh: 491: pid: parameter not set FAIL xiaoxia-worker-staging: not created ERROR: 1 个容器启动失败 → 自动回滚 ``` ## 根因 `scripts/ci_staging_deploy.sh` 开启了 `set -eu`。worker 容器的 `--health-cmd` 使用双引号包裹： ```sh --health-cmd "sh -c "for pid in /proc/[0-9]*/cmdline; do if grep -ql celery "$pid" ... ``` 其中的 **$pid** 被外层部署脚本在 `docker run` 参数展开阶段提前求值。`pid` 在该作用域未定义，`set -u` 下直接报 `parameter not set` 并终止，worker 容器未创建 → 部署判定失败 → 回滚到旧版本。 该 health-cmd 是 #1803 周期 Worker 健康检查修复（commit e4ac3983）引入。 ## 修复 将 health-cmd 内 `$pid` 转义为 `\$pid`，透传到容器内由 healthcheck shell 展开： ```sh --health-cmd "sh -c "for pid in /proc/[0-9]*/cmdline; do if grep -ql celery "\$pid" ... ``` - 主部署段（504 行）与回滚段（195 行）两处一并修复 - 已用 bash -n 语法检查 + 模拟 set -eu 展开验证：外层不再报错，容器内收到的 health-cmd 正确保留字面量 $pid 此修复同时解除 #1805（AI数字人 v3）合入后的 staging 部署阻塞。 🤖 Generated with [Claude Code](https://claude.com/claude-code)

xiaoxia added 1 commit 2026-09-08 22:17:14 +08:00

fix: 部署脚本 worker health-cmd 中 $pid 未转义导致 staging 部署失败回滚 ... cb3bc04c83

ci_staging_deploy.sh 开启 set -eu，worker 容器 --health-cmd 双引号内的
$pid 被外层部署脚本提前展开（pid 未定义）→ 'pid: parameter not set' →
worker 容器 docker run 失败 → 判定部署失败并自动回滚。

将 health-cmd 内 $pid 转义为 \$pid，使其透传到容器内由 healthcheck shell
展开（for pid in /proc/[0-9]*/cmdline）。回滚段(195行)与主部署段(504行)
两处一并修复。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

gitea-actions bot commented 2026-09-08 22:19:11 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia merged commit 6ee99a652c into develop 2026-09-08 22:41:28 +08:00

gitea-actions bot commented 2026-09-08 22:42:36 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1806 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1806