diff --git a/push-image.sh b/push-image.sh index bad87c7..f3f98f5 100755 --- a/push-image.sh +++ b/push-image.sh @@ -99,15 +99,27 @@ for arch in amd64 arm64; do docker push "$REPO:$TAG-$arch" || die "push failed for $arch" done -docker manifest rm "$REPO:$TAG" >/dev/null 2>&1 || true -docker manifest create "$REPO:$TAG" --amend "$REPO:$TAG-amd64" --amend "$REPO:$TAG-arm64" -docker manifest push "$REPO:$TAG" +# `docker manifest create` REFUSES a source that is itself a manifest list: +# ERROR: gitea.langenoja.se/...:-amd64 is a manifest list +# With Docker's containerd image store, `docker build --platform linux/amd64` +# produces an OCI index rather than a plain manifest, so both per-arch tags are +# lists and the combine step fails after the (slow, large) pushes have already +# succeeded — leaving the per-arch tags published and the plain tag absent, +# which is the one the Deployments reference. Seen on 12.11.1-ci6, 2026-08-31. +# +# `buildx imagetools create` accepts indexes as sources and does the combine +# registry-side, so it works with either image store. +docker buildx imagetools create -t "$REPO:$TAG" "$REPO:$TAG-amd64" "$REPO:$TAG-arm64" \ + || die "failed to combine the per-arch tags into $REPO:$TAG" # Verify by reading it back, rather than trusting the push's exit code. echo "verifying the published manifest lists both architectures" -got=$(docker manifest inspect "$REPO:$TAG" | grep -o '"architecture": *"[a-z0-9]*"' | sort -u | wc -l) -[ "$got" -ge 2 ] || die "published manifest does not list two architectures" -docker manifest inspect "$REPO:$TAG" | grep -o '"architecture": *"[a-z0-9]*"' | sort -u +inspect=$(docker buildx imagetools inspect "$REPO:$TAG") \ + || die "published $REPO:$TAG cannot be read back" +got=$(echo "$inspect" | grep -o 'linux/[a-z0-9]*' | sort -u | grep -cE 'linux/(amd64|arm64)') +[ "$got" -ge 2 ] || die "published manifest does not list both architectures: +$inspect" +echo "$inspect" | grep -o 'linux/[a-z0-9]*' | sort -u cat <