Skip to content

add: linux/arm/v7 to build platforms#843

Closed
nevil-mathew wants to merge 1 commit into
masterfrom
update-github-work
Closed

add: linux/arm/v7 to build platforms#843
nevil-mathew wants to merge 1 commit into
masterfrom
update-github-work

Conversation

@nevil-mathew

@nevil-mathew nevil-mathew commented Oct 15, 2025

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Chores
    • Extended Docker image platform support by adding linux/arm/v7 to the existing linux/amd64 and linux/arm64 builds. This enables running the container on 32-bit ARM devices (e.g., certain Raspberry Pi models) without additional steps. Container behavior and features remain unchanged; this update only broadens where the image can be used. No action is required for current users.

@coderabbitai

coderabbitai Bot commented Oct 15, 2025

Copy link
Copy Markdown

Walkthrough

The workflow for building Docker images was updated to include linux/arm/v7 in the multi-platform build matrix alongside linux/amd64 and linux/arm64. No other steps, logic, or control flow were modified.

Changes

Cohort / File(s) Summary
GitHub Actions workflow
\.github/workflows/docker-image.yml
Add linux/arm/v7 to Docker Buildx platforms; retain existing linux/amd64 and linux/arm64; no other workflow changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

A whisk of YAML, a hop through the vines,
I bundle my images for many designs.
From amd64 to arm’s little den,
Now v7 joins the burrow again.
Thump-thump—multi-arch dreams align! 🐇🚀

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly captures the primary change of adding linux/arm/v7 as a build platform in the CI workflow, directly reflecting the modifications made to docker-image.yml; it is clear, specific, and concise.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-github-work

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f7d7bf3 and 595ded9.

📒 Files selected for processing (1)
  • .github/workflows/docker-image.yml (1 hunks)

tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}
platforms: linux/amd64,linux/arm64
platforms: linux/amd64,linux/arm64,linux/arm/v7

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Add QEMU emulation before building for linux/arm/v7.

Expanding to linux/arm/v7 needs binfmt support. Unlike amd64/arm64, GitHub-hosted runners don’t have 32-bit ARM emulation preloaded, so BuildKit will fail when it tries to execute RUN layers for that architecture. Please add a docker/setup-qemu-action@v3 step (with platforms: arm,arm64) before invoking Buildx so the new target can actually build.

🤖 Prompt for AI Agents
.github/workflows/docker-image.yml around line 80: adding linux/arm/v7 requires
QEMU binfmt registration on GitHub runners; insert a docker/setup-qemu-action@v3
step (configure with platforms: arm,arm64) before the Buildx/build steps so QEMU
emulation is registered, then proceed to the existing buildx setup and build
with platforms: linux/amd64,linux/arm64,linux/arm/v7.

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