[PR #20] [MERGED] fix: stabilize docker build workflow for digest pushes #20

New Issue

2026-02-16T09:24:51-05:00

yindo commented

2026-02-16 09:24:51 -05:00

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-agentbox/pull/20
Author: @Mairuis
Created: 2/16/2026
Status: ✅ Merged
Merged: 2/16/2026
Merged by: @Mairuis

Base: main ← Head: fix/ci-separate-verify-and-push

📝 Commits (3)

96b15bf fix: split build into local verify and push steps to avoid concurrent export conflict
ec326d7 fix: separate PR verify build from push-by-digest flow
05b1e6e fix: run verification with explicit matrix platform

📊 Changes

1 file changed (+26 additions, -5 deletions)

View changed files

📝 .github/workflows/build.yml (+26 -5)

📄 Description

Summary

split the build path into two event-specific flows: PR builds load locally for verification, push/workflow_dispatch builds push by digest for publish
remove concurrent type=docker + type=image exporting in push runs, which caused intermittent Docker Hub blob upload failures
run verification with explicit matrix platform to avoid amd64/arm64 manifest mismatch during docker run

Why

Push runs were failing when Buildx exported local and registry outputs together, and verification could fail on mismatched runner/image platforms. This change keeps verification and publishing reliable while preserving Docker Hub + GHCR manifest publishing in the merge job.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/langgenius/dify-agentbox/pull/20 **Author:** [@Mairuis](https://github.com/Mairuis) **Created:** 2/16/2026 **Status:** ✅ Merged **Merged:** 2/16/2026 **Merged by:** [@Mairuis](https://github.com/Mairuis) **Base:** `main` ← **Head:** `fix/ci-separate-verify-and-push` --- ### 📝 Commits (3) - [`96b15bf`](https://github.com/langgenius/dify-agentbox/commit/96b15bf9be49443d393f06214ca420cf4a3635a7) fix: split build into local verify and push steps to avoid concurrent export conflict - [`ec326d7`](https://github.com/langgenius/dify-agentbox/commit/ec326d7ead11219716d61da60c8a498ac88180e7) fix: separate PR verify build from push-by-digest flow - [`05b1e6e`](https://github.com/langgenius/dify-agentbox/commit/05b1e6e233e65d8dd3b44aaaab6935587949bc39) fix: run verification with explicit matrix platform ### 📊 Changes **1 file changed** (+26 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/build.yml` (+26 -5) </details> ### 📄 Description ## Summary - split the build path into two event-specific flows: PR builds load locally for verification, push/workflow_dispatch builds push by digest for publish - remove concurrent `type=docker` + `type=image` exporting in push runs, which caused intermittent Docker Hub blob upload failures - run verification with explicit matrix platform to avoid amd64/arm64 manifest mismatch during `docker run` ## Why Push runs were failing when Buildx exported local and registry outputs together, and verification could fail on mismatched runner/image platforms. This change keeps verification and publishing reliable while preserving Docker Hub + GHCR manifest publishing in the merge job. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

yindo added the pull-request label 2026-02-16 09:24:51 -05:00

yindo closed this issue

2026-02-16 09:24:51 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: langgenius/dify-agentbox#20