fix(ci): staging-e2e/api-tests容器名改用GITHUB_SHA命名，彻底解决DooD模式命名冲突 #808

Merged

xiaoxia merged 1 commits from fix/staging-e2e-container-name into develop 2026-07-24 11:26:06 +08:00

Conversation 2 Commits 1 Files Changed 1

+2 -2

xiaoxia commented 2026-07-24 09:36:00 +08:00

Owner

Copy Link

Copy Source

问题

DooD模式下，CI容器的PID在同一Runner上是固定的（每个job容器PID=1），staging-e2e-$$ 命名方式会导致上一次失败没清理干净时，下一次同名冲突。

修复

• staging-e2e-$$ → staging-e2e-${GITHUB_SHA::8}
• staging-api-tests-$$ → staging-api-tests-${GITHUB_SHA::8}

用commit SHA前缀命名，天然唯一且有辨识度。配合已有的 docker rm -f 前置清理（#795），双重保障。

## 问题 DooD模式下，CI容器的PID在同一Runner上是固定的（每个job容器PID=1），`staging-e2e-$$` 命名方式会导致上一次失败没清理干净时，下一次同名冲突。 ## 修复 - `staging-e2e-$$` → `staging-e2e-${GITHUB_SHA::8}` - `staging-api-tests-$$` → `staging-api-tests-${GITHUB_SHA::8}` 用commit SHA前缀命名，天然唯一且有辨识度。配合已有的 `docker rm -f` 前置清理（#795），双重保障。

xiaoxia added 1 commit 2026-07-24 09:36:00 +08:00

fix(ci): staging-e2e/api-tests容器名改用GITHUB_SHA而非PID命名，彻底解决DooD模式下PID固定导致的命名冲突 d6f0cff195

auto-approve-bot commented 2026-07-24 09:44:35 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #808

⚠️ 问题（2个需要修改）

1. .gitea/workflows/ci-pipeline.yml 第1091行：容器命名存在并发冲突风险，可能导致CI任务失败。使用 GITHUB_SHA::8 作为唯一标识符，当同一个 Commit 触发多次 CI 任务（例如手动重试、不同分支同时指向同一 Commit）时，会产生相同的容器名称。后启动的任务会执行 docker rm -f 强制删除前序任务的容器，导致前序任务异常失败，或者 docker create 因名称占用而报错。
2. .gitea/workflows/ci-pipeline.yml 第1156行：容器命名存在并发冲突风险，同上。

💡 建议（1个可选）

1. 建议保留原有的进程ID $$ 或结合 GITHUB_SHA 使用，例如 CONTAINER_NAME="staging-e2e-${GITHUB_SHA::8}-$$"，既能关联提交信息，又能保证同一并发环境下的容器名称唯一性。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 01:44:35 | 模型:}

## 代码审查结果 - PR #808 ### ⚠️ 问题（2个需要修改） 1. **.gitea/workflows/ci-pipeline.yml 第1091行**：容器命名存在并发冲突风险，可能导致CI任务失败。使用 `GITHUB_SHA::8` 作为唯一标识符，当同一个 Commit 触发多次 CI 任务（例如手动重试、不同分支同时指向同一 Commit）时，会产生相同的容器名称。后启动的任务会执行 `docker rm -f` 强制删除前序任务的容器，导致前序任务异常失败，或者 `docker create` 因名称占用而报错。 2. **.gitea/workflows/ci-pipeline.yml 第1156行**：容器命名存在并发冲突风险，同上。 ### 💡 建议（1个可选） 1. 建议保留原有的进程ID `$$` 或结合 `GITHUB_SHA` 使用，例如 `CONTAINER_NAME="staging-e2e-${GITHUB_SHA::8}-$$"`，既能关联提交信息，又能保证同一并发环境下的容器名称唯一性。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 01:44:35 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 907cc63f96 into develop 2026-07-24 11:26:06 +08:00

xiaoxia deleted branch fix/staging-e2e-container-name 2026-07-24 11:26:06 +08:00

gitea-actions bot commented 2026-07-24 11:45:21 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #808 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#808