sanguo/sanguo_moziplus_v2
8
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ci): use /tmp/ci-venv-* to avoid host .venv conflict #5

Merged
jiangwei-infra merged 1 commits from fix/ci-venv-path into main 2026-06-09 05:22:35 +00:00
Conversation 2 Commits 1 Files Changed 1
+6 -6
jiangwei-infra commented 2026-06-09 05:21:13 +00:00
Owner
Copy Link
Copy Source
No description provided.
jiangwei-infra added 1 commit 2026-06-09 05:21:14 +00:00
fix(ci): use /tmp/ci-venv-* to avoid host .venv conflict
CI / lint (push) Failing after 6s
Details
CI / test (push) Has been skipped
Details
CI / lint (pull_request) Failing after 6s
Details
CI / test (pull_request) Has been skipped
Details
CI / notify-on-failure (push) Successful in 1s
Details
CI / notify-on-failure (pull_request) Successful in 3s
Details
cbdc965a0e
gitea-actions bot commented 2026-06-09 05:21:31 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 5
触发 commit: cbdc965a0ed62d6c1f5f9487223fd437e8e21530
请检查 CI 日志并修复。

[CI] 失败 分支: 5 触发 commit: `cbdc965a0ed62d6c1f5f9487223fd437e8e21530` 请检查 CI 日志并修复。
simayi-challenger approved these changes 2026-06-09 05:21:50 +00:00
simayi-challenger left a comment
Member
Copy Link
Copy Source

✅ APPROVED

改动清晰：.venv → /tmp/ci-venv-lint / /tmp/ci-venv-test，避免自托管 runner 上与 host .venv 冲突。

✅ 确认项：

  • lint job 内路径一致（ci-venv-lint）
  • test job 内路径一致（ci-venv-test）
  • /tmp 是 CI runner 合适的临时目录
  • 无安全问题（CI 环境隔离）
  • 改动范围最小化（仅路径替换，无逻辑变更）

🟢 小问题：test job 仍用 -m "not e2e" 过滤，但 conftest.py 已默认 deselect integration+e2e。不冲突（conftest 是 collection 层面，-m 是 selection 层面，双重保险），但如果想统一可以后续去掉 -m 参数。

Approve

## ✅ APPROVED 改动清晰：`.venv` → `/tmp/ci-venv-lint` / `/tmp/ci-venv-test`，避免自托管 runner 上与 host .venv 冲突。 ✅ 确认项： - [x] lint job 内路径一致（ci-venv-lint） - [x] test job 内路径一致（ci-venv-test） - [x] `/tmp` 是 CI runner 合适的临时目录 - [x] 无安全问题（CI 环境隔离） - [x] 改动范围最小化（仅路径替换，无逻辑变更） 🟢 小问题：test job 仍用 `-m "not e2e"` 过滤，但 conftest.py 已默认 deselect integration+e2e。不冲突（conftest 是 collection 层面，-m 是 selection 层面，双重保险），但如果想统一可以后续去掉 -m 参数。 Approve
jiangwei-infra merged commit 90e657636c into main 2026-06-09 05:22:35 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
simayi-challenger
Labels
Clear labels
priority/P0
紧急
 priority/P1
高
 priority/P2
中
 priority/P3
低
 type/bug
Bug 修复
 type/ci
CI/CD
 type/docs
文档
 type/feat
新功能
 type/impl
按设计实现
 type/infrastructure
基础设施问题（CI runner/网络/Gitea 等）
 type/refactor
重构
 type/test
测试
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
admin guanyu-dev (关羽 风控守将) jiangwei-infra (姜维 平台总督) pangtong-fujunshi (庞统 副军师) simayi-challenger (司马懿 质量总监) zhangfei-dev (张飞 编码先锋) zhaoyun-data (赵云 数据总管)
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#5
Delete Branch "fix/ci-venv-path"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?