Skip to content

revert: use amd64-only docker builds instead of multi-arch#811

Merged
adityachoudhari26 merged 1 commit intomainfrom
single-platform-only
Feb 19, 2026
Merged

revert: use amd64-only docker builds instead of multi-arch#811
adityachoudhari26 merged 1 commit intomainfrom
single-platform-only

Conversation

@adityachoudhari26
Copy link
Member

@adityachoudhari26 adityachoudhari26 commented Feb 19, 2026

Summary by CodeRabbit

  • Chores
    • Updated build configuration to specify explicit platform targets for containerized services, improving build consistency and reliability across deployment pipelines.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

Six GitHub Actions workflow files are updated to explicitly specify the Docker build platform as linux/amd64 across all application and package builds. Each modification adds a single platform configuration parameter to the respective build steps.

Changes

Cohort / File(s) Summary
Docker Build Platform Specification
.github/workflows/apps-api.yaml, apps-relay.yaml, apps-web.yaml, apps-workspace-engine-router.yaml, apps-workspace-engine.yaml, packages-migrations.yaml
Added explicit platform: "linux/amd64" configuration to Docker image build steps across all workflows to standardize the target build platform.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Six workflows aligned in perfect harmony,
Each Docker build now knows its destiny—
linux/amd64, a steadfast ground,
No more surprise builds spinning 'round! 🐳✨

🚥 Pre-merge checks | ✅ 3
✅ 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 clearly describes the main change: adding platform constraints to Docker builds, converting from multi-architecture builds to amd64-only. This accurately summarizes the consistent modification across all six workflow files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch single-platform-only

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.

@adityachoudhari26 adityachoudhari26 merged commit 31f06c3 into main Feb 19, 2026
14 checks passed
@adityachoudhari26 adityachoudhari26 deleted the single-platform-only branch February 19, 2026 04:58
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