fix(ci): restore proper Docker daemon gate on publish-workspace-server-image

main merged a fix (3206966e) that replaces the broken `Diagnose Docker
daemon access` step (|| true guards) with a proper `Verify Docker daemon
access` gate (docker info || { exit 1 }). The feature branch is still on
the old broken version — sync it.

mc#711: ubuntu-latest runners may lack a live Docker daemon. With the
old guards the step always succeeded even when Docker was inaccessible,
letting the build step hang for 4+ minutes before failing. The restored
gate fails in ~5s with an actionable error message.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
Molecule AI · infra-lead 2026-05-13 22:29:04 +00:00
parent 4ac35fab53
commit bf41b18de9

View File

@ -65,20 +65,22 @@ jobs:
- name: Checkout
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
- name: Diagnose Docker daemon access
# Health check: verify Docker daemon is accessible before attempting any
# build steps. This fails loudly at step 1 when the runner's docker.sock
# is inaccessible rather than silently continuing where `docker build`
# fails deep in the process with a cryptic ECR auth error.
- name: Verify Docker daemon access
run: |
set -euo pipefail
echo "::group::Docker daemon diagnosis"
echo "::group::Docker daemon health check"
echo "Runner: ${HOSTNAME:-unknown}"
echo "--- Socket info ---"
ls -la /var/run/docker.sock 2>/dev/null || echo "/var/run/docker.sock: not found"
stat /var/run/docker.sock 2>/dev/null || true
echo "--- User info ---"
id
echo "--- docker version ---"
docker version 2>&1 || true
echo "--- docker info (full) ---"
docker info 2>&1 || echo "docker info failed: exit $?"
docker info 2>&1 | head -5 || {
echo "::error::Docker daemon is not accessible at /var/run/docker.sock"
echo "::error::Runner: ${HOSTNAME:-unknown}"
echo "::error::Check: (1) daemon is running, (2) runner user is in docker group, (3) sock permissions are 660+"
exit 1
}
echo "Docker daemon OK"
echo "::endgroup::"
# Pre-clone manifest deps before docker build.