Skip to content

Conversation

@TonyRL
Copy link
Collaborator

@TonyRL TonyRL commented Dec 27, 2025

Involved Issue / 该 PR 相关 Issue

Close #

Example for the Proposed Route(s) / 路由地址示例

NOROUTE

Note / 说明

docs: https://docs.docker.com/build/ci/github-actions/multi-platform/#distribute-build-across-multiple-runners
This greatly reduces build time (both cached and non-cached)
Image

Copilot AI review requested due to automatic review settings December 27, 2025 00:06
@vercel

This comment was marked as off-topic.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the Docker build workflow to use native ARM64 runners instead of QEMU emulation for multi-platform builds, significantly reducing build times for both cached and non-cached builds.

Key changes:

  • Introduces a matrix strategy with native runners (ubuntu-latest for AMD64, ubuntu-24.04-arm for ARM64)
  • Implements digest-based multi-platform image builds with a separate merge job to create manifest lists
  • Migrates DOCKER_USERNAME from secrets to variables and adds dynamic repository name extraction

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
.github/workflows/docker-release.yml Main workflow refactored to use native runners with matrix strategy, digest-based builds, and a new merge job for manifest creation. Replaces cosign signing with GitHub attestations
.github/workflows/docker-test.yml Updates cache scope and DOCKER_USERNAME reference to align with the new platform-specific caching strategy

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@TonyRL TonyRL merged commit 85d36a2 into DIYgod:master Dec 27, 2025
31 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant