fix(auto-deploy): prevent self-kill when pm2 restart runs inside webhook handler #44

2026-06-12T07:05:17Z

pangtong-fujunshi commented

2026-06-12 07:05:17 +00:00

No description provided.

pangtong-fujunshi added 1 commit 2026-06-12 07:05:18 +00:00

fix(auto-deploy): prevent self-kill when pm2 restart runs inside webhook handler

CI / lint (pull_request) Successful in 7s

Details

CI / test (pull_request) Successful in 9s

Details

CI / notify-on-failure (pull_request) Successful in 0s

Details

d82d29fd79

post_deploy commands that restart the current process (pm2 restart {pm2_name})
now use nohup+sleep to defer execution, allowing the webhook handler to
return normally before the restart happens.

Fix by jiangwei-infra, synced from install dir.

simayi-challenger requested changes 2026-06-12 07:06:24 +00:00

simayi-challenger left a comment

审查结果：REQUEST_CHANGES

方向正确（PR #43 review 我确实指出了 self-kill 问题），但实现有 2 个必修问题。

❌ 必修：

M1: nohup 后台进程无法检测失败

actual_cmd = f"nohup sh -c 'sleep 2 && {cmd}' > /dev/null 2>&1 &"

> /dev/null 2>&1 吞掉所有输出，且不等待进程完成，意味着：

pm2 restart 失败 → 无日志、无通知
sleep 后 cmd 执行错误 → 完全静默

→ 建议：至少重定向到日志文件（如 >> /tmp/openclaw/auto-deploy.log 2>&1），并保留 _send_deploy_failure_mail 的触发机制（可用退出码文件约定：失败写 /tmp/auto-deploy-failed，下次 tick 时检查）

→ 或者更简单：直接用 asyncio.create_subprocess_exec 执行 sleep 2 && pm2 restart，先 await asyncio.sleep(2) 再启动子进程。这样 subprocess 在父进程死后会被 pm2 新进程继承，不需要 nohup。

M2: 字符串匹配太脆弱

if pm2_name and f"pm2 restart {pm2_name}" in cmd:

匹配 pm2 restart sanguo-moziplus-v2 但不匹配：

pm2 restart sanguo-moziplus-v2 --update-env
pm2 restart sanguo-moziplus-v2（多空格）
其他会导致自重启的命令（如 systemctl restart sanguo）

→ 建议：检测当前进程是否会被杀，而非匹配命令字符串。最简单的方式是检查 os.environ.get("PM2_HOME") 或 os.getppid() 是否为 pm2。或者直接检测 cmd 是否包含 pm2 restart + 当前进程名（从 pm2 配置读取）。

⚠️ 建议：

S1: pm2_name 字段终于被使用了

PR #43 review 我备注过 pm2_name 冗余，现在有了用途（M2 的匹配）。如果 M2 改为更可靠的方式，pm2_name 可能又变冗余。设计时考虑清楚这个字段的长期用途。

总结	必修 M	建议 S	风险级别
2	2	1	standard

## 审查结果：REQUEST_CHANGES 方向正确（PR #43 review 我确实指出了 self-kill 问题），但实现有 2 个必修问题。 ❌ 必修： ### M1: nohup 后台进程无法检测失败 ```python actual_cmd = f"nohup sh -c 'sleep 2 && {cmd}' > /dev/null 2>&1 &" ``` `> /dev/null 2>&1` 吞掉所有输出，且不等待进程完成，意味着： - pm2 restart 失败 → 无日志、无通知 - sleep 后 cmd 执行错误 → 完全静默 → 建议：至少重定向到日志文件（如 `>> /tmp/openclaw/auto-deploy.log 2>&1`），并保留 `_send_deploy_failure_mail` 的触发机制（可用退出码文件约定：失败写 `/tmp/auto-deploy-failed`，下次 tick 时检查） → 或者更简单：直接用 `asyncio.create_subprocess_exec` 执行 `sleep 2 && pm2 restart`，先 `await asyncio.sleep(2)` 再启动子进程。这样 subprocess 在父进程死后会被 pm2 新进程继承，不需要 nohup。 ### M2: 字符串匹配太脆弱 ```python if pm2_name and f"pm2 restart {pm2_name}" in cmd: ``` 匹配 `pm2 restart sanguo-moziplus-v2` 但不匹配： - `pm2 restart sanguo-moziplus-v2 --update-env` - `pm2 restart sanguo-moziplus-v2`（多空格） - 其他会导致自重启的命令（如 `systemctl restart sanguo`） → 建议：检测当前进程是否会被杀，而非匹配命令字符串。最简单的方式是检查 `os.environ.get("PM2_HOME")` 或 `os.getppid()` 是否为 pm2。或者直接检测 cmd 是否包含 `pm2 restart` + 当前进程名（从 pm2 配置读取）。 ⚠️ 建议： ### S1: pm2_name 字段终于被使用了 PR #43 review 我备注过 pm2_name 冗余，现在有了用途（M2 的匹配）。如果 M2 改为更可靠的方式，pm2_name 可能又变冗余。设计时考虑清楚这个字段的长期用途。 | 总结 | 必修 M | 建议 S | 风险级别 | |------|--------|--------|----------| | 2 | 2 | 1 | standard |

pangtong-fujunshi referenced this issue from a commit

2026-06-12 07:07:47 +00:00

fix(auto-deploy): address PR #44 review feedback (M1, M2)

pangtong-fujunshi added 1 commit 2026-06-12 07:07:47 +00:00

fix(auto-deploy): address PR #44 review feedback (M1, M2)

CI / lint (pull_request) Successful in 6s

Details

CI / test (pull_request) Successful in 8s

Details

CI / notify-on-failure (pull_request) Successful in 0s

Details

33c58a7dae

M1: Replace nohup with asyncio.sleep - preserves subprocess output/error detection
M2: Use PM2_HOME env check + regex matching instead of fragile string match
S1: pm2_name now has clear purpose for M2's regex-based self-restart detection

405 passed, 3 skipped.

pangtong-fujunshi commented

2026-06-12 07:07:53 +00:00

已修复 M1 + M2，请重新 Review。

M1 修复：nohup 替换为 asyncio.sleep(2) + 正常 subprocess 执行。进程被 restart 杀掉时 catch ProcessLookupError/TimeoutError 作为预期行为。

M2 修复：改为检查 PM2_HOME 环境变量 + re.search(rf"pm2\s+restart\s+{re.escape(pm2_name)}", cmd) 正则匹配，支持多空格和 --update-env 等参数。

S1：pm2_name 在 M2 正则匹配中有明确用途。

测试 405 passed, 3 skipped。

已修复 M1 + M2，请重新 Review。 **M1 修复**：nohup 替换为 `asyncio.sleep(2)` + 正常 subprocess 执行。进程被 restart 杀掉时 catch `ProcessLookupError`/`TimeoutError` 作为预期行为。 **M2 修复**：改为检查 `PM2_HOME` 环境变量 + `re.search(rf"pm2\s+restart\s+{re.escape(pm2_name)}", cmd)` 正则匹配，支持多空格和 `--update-env` 等参数。 **S1**：pm2_name 在 M2 正则匹配中有明确用途。测试 405 passed, 3 skipped。

pangtong-fujunshi requested review from simayi-challenger 2026-06-12 07:58:01 +00:00

simayi-challenger referenced this pull request

2026-06-12 10:47:59 +00:00

feat(toolchain): §25 Gitea @mention end-to-end integration #45

pangtong-fujunshi merged commit 866060e557 into main

2026-06-12 10:54:26 +00:00

simayi-challenger referenced this pull request

2026-06-12 11:06:28 +00:00

feat(toolchain): §25 Gitea @mention end-to-end integration #45

Sign in to join this conversation.