Skip to content

feat(build): add arm64 support with cross-compilation#8630

Closed
gustavovalverde wants to merge 2 commits intomainfrom
feat-arm64-build
Closed

feat(build): add arm64 support with cross-compilation#8630
gustavovalverde wants to merge 2 commits intomainfrom
feat-arm64-build

Conversation

@gustavovalverde
Copy link
Member

@gustavovalverde gustavovalverde commented Jun 20, 2024

Motivation

To run dockerized containers in M1 and other ARM64 processors, we need to cross-compile the Docker image for this platforms. This makes it very hard to test locally, as all builds must be done remotely, or pushing to confirm on GHA.

This supersedes #3678
And reverts: #3791

Solution

Previously, we had to temporarily remove ARM64 support for Zebra until we could improve build times or have ARM64 instances to build it. Now that we're using Docker Build Cloud, we don't have to deploy ARM64 runners specifically for this task.

Tests

  • Download the resulting artifact to an ARM64 machine and run it with docker, without using CPU virtualization.

Follow-up Work

  • Must be done here: Update documentation for supported platforms.

PR Author's Checklist

  • The PR name will make sense to users.
  • The PR provides a CHANGELOG summary.
  • [] The solution is tested.
  • The documentation is up to date.
  • The PR has a priority label.

PR Reviewer's Checklist

  • The PR Author's checklist is complete.
  • The PR resolves the issue.

@gustavovalverde gustavovalverde added A-docs Area: Documentation A-devops Area: Pipelines, CI/CD and Dockerfiles I-usability Zebra is hard to understand or use C-feature Category: New features A-release Area: Zebra releases and release management P-Low ❄️ labels Jun 20, 2024
@gustavovalverde gustavovalverde self-assigned this Jun 20, 2024
@github-actions github-actions bot added the C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG label Jun 20, 2024
@upbqdn upbqdn added the do-not-merge Tells Mergify not to merge this PR label Jun 28, 2024
@arya2 arya2 removed the do-not-merge Tells Mergify not to merge this PR label Jul 2, 2024
@arya2 arya2 added do-not-merge Tells Mergify not to merge this PR and removed do-not-merge Tells Mergify not to merge this PR labels Aug 27, 2024
@gustavovalverde
Copy link
Member Author

We won't be able to do this until zcash/lightwalletd#476 is merged, and we don't have a deadline for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-devops Area: Pipelines, CI/CD and Dockerfiles A-docs Area: Documentation A-release Area: Zebra releases and release management C-feature Category: New features C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG I-usability Zebra is hard to understand or use P-Low ❄️

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants