Skip to content

feat(docker): Add support for multiple platforms in Docker action#675

Open
gondoi wants to merge 2 commits into
mainfrom
feature/devops-1248-enable-multi-arch-build-containers-group-1
Open

feat(docker): Add support for multiple platforms in Docker action#675
gondoi wants to merge 2 commits into
mainfrom
feature/devops-1248-enable-multi-arch-build-containers-group-1

Conversation

@gondoi
Copy link
Copy Markdown

@gondoi gondoi commented Apr 20, 2026

Summary

Checklist

  • I've added error handling for all actions/requests, and verified how this error will show on UI. (or there was no error handling)
  • I've reviewed all the copy changed/added in this PR, using AI if needed. (or there was no copy changes)
  • I've included the reference to the issues being closed from Github and/or Linear (or there was no issues)
  • I've changed the Docs to reflect my changes (or it was not needed)
  • I've put docs links where it may be helpful (or it was not needed)
  • I checked the resulting UI both in Light and Dark mode (or no UI changes were made)
  • I reviewed the entire PR myself (preferably, on GH UI)

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fuel-explorer-v2-vite Ready Ready Preview May 27, 2026 3:47pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 20, 2026

PR Summary

Medium Risk
Changes how production images are built and pushed (multi-runner native builds); misconfiguration could break releases or PR image builds on ghcr.io.

Overview
Docker image publishing is moved from an in-repo composite action to the shared FuelLabs/github-actions reusable workflow docker-build-push.yml, and the local .github/actions/docker-publish action is removed.

The docker-publish workflow now declares packages: write and id-token: write, passes registry credentials and build inputs (including linux/amd64 and linux/arm64, separate amd64/arm64 runners, and the same tag patterns) to the reusable workflow instead of checking out and invoking the composite action on a single runner.

Reviewed by Cursor Bugbot for commit 13b63bd. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 79dbf83. Configure here.

Comment thread .github/actions/docker-publish/action.yaml Outdated
Comment thread .github/actions/docker-publish/action.yaml Outdated
Migrate docker-publish to FuelLabs/github-actions docker-build-push
with build-backend native and linux/amd64,linux/arm64 on native runners.

Refs DEVOPS-1248

Co-authored-by: Cursor <cursoragent@cursor.com>
@gondoi gondoi force-pushed the feature/devops-1248-enable-multi-arch-build-containers-group-1 branch from 801a8a9 to 7ed4257 Compare May 27, 2026 15:27
Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant