[codex] Fix Watchtower Docker API version for Docker 29#81
[codex] Fix Watchtower Docker API version for Docker 29#81ZeroPointSix wants to merge 5 commits into
Conversation
|
总体结论:这个 PR 是小范围配置兼容修复,目标明确,能解决 Docker Engine 29+ 下 Watchtower 默认 Docker API 版本过低的问题;未发现阻塞级问题。 关键发现:
优先级建议:
后续建议:
|
|
CodeXWeb 本轮自动巡检已认领,正在处理本 PR 的 review 建议。 本轮只会处理这一项:#81。 处理范围限定为 review comment 4670506235 中的合并前验证建议:核对 Watchtower |
|
|
CodeXWeb 本轮自动巡检处理结果:本轮只处理 PR #81,未切换到其他 Issue/PR,也不会自动合并。 已复核维护者评论 #81 (comment) 中关于 Watchtower Docker API 版本兼容性的建议。 依据:
验证结果:
完整 pytest 失败项:
限制:
留痕:
|
ZeroPointSix
left a comment
There was a problem hiding this comment.
总体结论:本轮增量审查覆盖 PR #81 当前 head 22d13edd86385b0e4f3bae18255e4254b5267d06。配置修复方向此前已被评论覆盖;本次新增发现是当前 Code Quality 仍失败,合并前需要先修正格式检查。
关键发现:
- 阻塞:GitHub Actions
Code Qualityworkflow 中Code Lintingjob 失败,失败步骤是Run Black (Code Formatter Check)。日志显示tests/test_watchtower_docker_api_config.py会被 Black 重新格式化,差异是from pathlib import Path与import unittest之间多了一个空行;因此当前 head 无法通过代码质量门禁。 - 影响:这不是 Watchtower Docker API 方案本身的问题,但会阻止 PR 以当前状态合并,也会让后续 reviewer 继续追踪同一个 CI 红灯。
- 建议:在 PR 分支运行
black tests/test_watchtower_docker_api_config.py,提交格式化结果后重新触发 Code Quality。该改动应只影响测试文件格式,不需要改业务逻辑或 Docker 配置。
优先级建议:
- 阻塞:先修复 Black 格式失败并确认
Code Quality变绿。 - 中:保留此前评论中的限制说明:当前沙箱没有 Docker/Compose,真实 Watchtower 容器级 smoke 仍需要在具备 Docker 的环境确认。
- 低:如果继续保留完整 pytest 的既有失败说明,建议和 Code Quality 状态分开记录,避免把格式门禁失败误归因到外部 E2E。
后续建议:
- 修复格式后,本 PR 的核心验证重点仍是:
DOCKER_API_VERSION=${WATCHTOWER_DOCKER_API_VERSION:-1.44}是否能在目标 Docker Engine 29.x 部署环境中让 Watchtower 稳定启动并暴露 HTTP API。
巡检审查标记:PR & Issue Reviewer 已审查当前 head commit 22d13edd86385b0e4f3bae18255e4254b5267d06;本轮只对 ZeroPointSix/outlookEmailPlus#81 写回增量审查意见。



Summary
DOCKER_API_VERSIONto1.44by default so it can connect to Docker Engine 29+, whose daemon rejects clients older than API v1.44.WATCHTOWER_DOCKER_API_VERSIONas an optional override in.env.examplefor operators who need to tune it.Fixes #79
Tests
PYTHONPATH=.pytest_deps python3 -m unittest tests.test_version_update -v(51 tests passed in the sandbox while developing the fix)