Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate multiarch Docker images #20848

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Generate multiarch Docker images #20848

wants to merge 8 commits into from

Conversation

dgadelha
Copy link

@dgadelha dgadelha commented Apr 8, 2024

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

This PR moves the build_containers.sh script to use Docker's buildx, with the multiarch image --platform flag enabled

Related Tickets & Documents

The forem/ruby (build_base_ruby_image.sh) seems to be already using buildx/multiarch, so this PR changes the quay.io/forem/forem images

QA Instructions, Screenshots, Recordings

It should be now possible to use container-compose.yml on ARM64 systems.

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above.

  • Yes
  • No, and this is why: CI/CD-related change, no code changed
  • I need help with writing tests

@dgadelha dgadelha requested review from a team as code owners April 8, 2024 15:42
@dgadelha dgadelha requested review from maestromac and lightalloy and removed request for a team April 8, 2024 15:42
Copy link
Contributor

github-actions bot commented Apr 8, 2024

Thank you for opening this PR! We appreciate you!

For all pull requests coming from third-party forks we will need to
review the PR before we can process it through our CI pipelines.

A Forem Team member will review this contribution and get back to
you as soon as possible!

Copy link
Contributor

github-actions bot commented Apr 8, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@dgadelha
Copy link
Author

dgadelha commented Apr 8, 2024

I have read the CLA Document and I hereby sign the CLA

Copy link
Contributor

github-actions bot commented Apr 10, 2024

Uffizzi Ephemeral Environment Deploying

☁️ https://app.uffizzi.com/github.com/forem/forem/pull/20848

⚙️ Updating now by workflow run 8816192168.

What is Uffizzi? Learn more!

@dgadelha
Copy link
Author

The crash on preview environment seems not to be related to the changes caused by this PR.

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.

Provide linux/arm64 architecture support with an open-source chain of images (eg. forem/ruby)
2 participants