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

Add support for building on ARM64 (and Apple Silicon). #182

Closed
wants to merge 1 commit into from

Conversation

chrismoos
Copy link

From my testing, I was able to build Confluent ZK and Kafka images using the changes here as the base image and it seemed to run fine, natively, on an Apple M1 machine.

This would resolve #117 and confluentinc/kafka-images#80.

NOTE: Probably the PR on this branch isn't the best, but I was unable to test locally against any newer versions as the various Confluent artifacts for versions > 7.0.1 were not found in https://packages.confluent.io/maven/.

Changes

Testing

I pushed the base, ZK, and Kafka images if you want to test:

  • chrismoos/cp-kafka:7.0.1
  • chrismoos/cp-base-new:7.0.1-ubi8
  • chrismoos/cp-zookeeper:7.0.1

* Update Zulu version to multiarch (ARM supported one)
* Remove dockerfile-maven-plugin building -- there is an upstream issue
  with building on ARM (FFI issue
  spotify/dockerfile-maven#367)
@andrewegel
Copy link
Contributor

Remove dockerfile-maven-plugin building -- there is an upstream issue with building on ARM (FFI issue spotify/dockerfile-maven#367)

Yeah this is why we can't take this change - This is too large of a refactor to take within a release. We are in the process of building ARM64 images using the dockerfile-maven plugin on native ARM64 systems to spit out ARM64 images - Its not the most efficient, but it doesn't require a large re-write like you're doing here.

@andrewegel andrewegel closed this Mar 7, 2022
@chrismoos
Copy link
Author

@andrewegel Okay, thanks for the details. Looking forward to some official builds from you guys then. :)

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.

2 participants