Skip to content
This repository was archived by the owner on Sep 2, 2026. It is now read-only.

fix: 禁止发布任务启动 Docker Desktop - #80

Merged
HyxiaoGe merged 1 commit into
masterfrom
fix/issue-78-windows-docker
Aug 31, 2026
Merged

HyxiaoGe merged 1 commit into
masterfrom
fix/issue-78-windows-docker

Conversation

@HyxiaoGe

Copy link
Copy Markdown
Owner

背景

Windows 服务型 Runner 从 Session 0 启动并保留 Docker Desktop,会占用后端命名管道并阻断桌面用户启动 Docker Desktop。关联 #78。

改动

  • 发布前仅检查 Runner 已预配置的 Docker Engine
  • Docker 不可用或无访问权限时立即失败
  • 删除 GUI 启动、服务启动、等待重试和 RUNNER_TRACKING_ID 绕过逻辑
  • 更新 CI/CD 权限边界测试,禁止危险逻辑回归

验证

  • python3.11 -m pytest test/test_ci_cd_permission_boundary.py test/test_ci_container_contract.py test/test_deploy_auth_config.py test/test_knowledge_deploy_config.py test/test_litellm_governance_units.py test/test_model_management_deploy_config.py -q:83 passed,14 subtests passed
  • python3.11 -m ruff check test/test_ci_cd_permission_boundary.py:通过
  • git diff --check:通过

发布约束

若 Windows Runner 或 Docker 出现异常,停止本轮发布,不重跑 CI、不循环排查 Windows、不改走临时发布路径。

背景:Windows 服务型 Runner 从 Session 0 启动并保留 Docker Desktop,会占用仅该会话可访问的后端管道并阻断桌面用户。

改动:发布前仅检查预配置 Docker Engine;不可用或无权限时立即失败,并新增契约测试禁止 GUI、服务启动、等待重试和 Runner 跟踪绕过。

验证:python3.11 -m pytest 关联配置测试通过 83 项及 14 个子测试;ruff 与 git diff --check 通过。

Co-Authored-By: Codex <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@HyxiaoGe
HyxiaoGe merged commit bf77ca5 into master Aug 31, 2026
1 check passed
@HyxiaoGe
HyxiaoGe deleted the fix/issue-78-windows-docker branch August 31, 2026 06:58
@HyxiaoGe
HyxiaoGe restored the fix/issue-78-windows-docker branch August 31, 2026 06:59
@HyxiaoGe
HyxiaoGe deleted the fix/issue-78-windows-docker branch August 31, 2026 07:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant