Skip to content

chore: 删心跳邮件告警 + docker healthcheck,只留 status 页 - #54

Merged
Color2333 merged 1 commit into
mainfrom
chore/remove-heartbeat-alert
Jul 18, 2026
Merged

chore: 删心跳邮件告警 + docker healthcheck,只留 status 页#54
Color2333 merged 1 commit into
mainfrom
chore/remove-heartbeat-alert

Conversation

@Color2333

Copy link
Copy Markdown
Owner

背景

心跳机制此前分3层:①docker healthcheck 自动重启卡死 worker ②status 页人看状态 ③邮件告警主动推送。按需求简化——worker 卡死不再自动重启/发邮件,只靠 status 页发现 + 人工介入。

删除

  • apps/worker/main.pyheartbeat_alert_job + _send_alert + _last_alerts + _ALERT_DEDUP_SECONDS(邮件告警去重)+ scheduler 注册
  • docker-compose.yml:worker healthcheck 段(不再自动重启)
  • scripts/worker_healthcheck.py:整个文件删除
  • tests/test_repositories.pyTestWorkerAlertDedup 2 测试(测已删的 _send_alert

保留

  • status 页/system/worker 端点 + Operations 心跳面板(读共享卷心跳展示时效)
  • last_error 面板:topic 抓取错误展示 + 重新抓取按钮(与心跳独立)
  • worker 写心跳逻辑:各 job 成功写心跳,status 页消费

调整

  • frontend Operations.tsx:过期提示从"worker 每 10min 自检发告警邮件"改"请检查 worker 容器状态与日志,必要时手动重启"

验证

  • ruff + tsc + 61 passed(删 2 个告警去重测试)
  • 无残留引用(grep 确认无 _send_alert/heartbeat_alert/worker_healthcheck 引用)

影响

  • worker 卡死不再自动重启——需人工看 status 页发现后介入(按需求接受此权衡)
  • 无新迁移、无配置变更

按需求简化心跳机制:worker 卡死不再自动重启/发邮件,靠 status 页
(/system/worker 端点 + Operations 面板)发现 + 人工介入。

删除:
- apps/worker/main.py:heartbeat_alert_job + _send_alert + _last_alerts
  + _ALERT_DEDUP_SECONDS(邮件告警去重)+ scheduler 注册。保留
  _write_heartbeat/_read_heartbeat/_HEALTH_FILE(status 页读)。
- docker-compose.yml:worker healthcheck 段(不再自动重启卡死 worker)。
- scripts/worker_healthcheck.py:整个文件删除(healthcheck 脚本)。
- tests/test_repositories.py:TestWorkerAlertDedup 2 测试(测已删的 _send_alert)。

调整:
- frontend Operations.tsx:过期提示从"worker 每 10min 自检发告警邮件"
  改"请检查 worker 容器状态与日志,必要时手动重启"。

保留:
- status 页(/system/worker 端点 + Operations 心跳面板)读共享卷心跳
  展示 worker 健康时效。
- last_error 面板(topic 抓取错误展示 + 重新抓取)与心跳机制独立,保留。
- worker 写心跳逻辑(各 job 成功写心跳,status 页消费)。

验证:ruff + tsc + 61 passed(删 2 个告警去重测试)。无残留引用。
@github-actions

Copy link
Copy Markdown

🔍 OpenCode PR Review Required

这是一个受保护的分支,merge 前需要进行 code review。

请运行以下命令进行 OpenCode review:

/oc review https://github.com/Color2333/PaperMind/pull/$PR_NUM

或者在 PR 页面评论 /oc 来触发 OpenCode review。


This is an automated reminder from PR Review Gate.

@Color2333
Color2333 merged commit b5b60dd into main Jul 18, 2026
2 checks passed
@Color2333
Color2333 deleted the chore/remove-heartbeat-alert branch July 18, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant