search

orientechnologies / orientdb-docker

OrientDB running in a docker container
MIT License
63 stars 63 forks source link

ARM64 support #48

Open odidev opened 6 years ago

odidev commented 6 years ago

OrientDB is working pretty well for arm64v8 architecture.

I would be happy to raise a PR if you allow me to do so.

Moreover, this doesn't even require changes in Dockerfile too. The only thing which is required is to either add a new folder for arm64v8 or just expose contents of x86_64 outside to be used for both x86_64 & arm64v8.

I can provide Build & Run logs as well for confirmation.

Regards,

luigidellaquila commented 6 years ago

Hi @odidev

Sure, it will be very appreciated!

Thanks

Luigi

odidev commented 6 years ago

@luigidellaquila Hi Luigi, Should I go for adding a new folder for arm64v8 or delete x86_64 folder?

Regards,

luigidellaquila commented 6 years ago

HI @odidev

I'd say let's add a new folder for ARM

Thanks

Luigi

odidev commented 6 years ago

@luigidellaquila - Raised PR as per your guidance :smile: Please have a look.

Regards,

lag-linaro commented 5 years ago

Still waiting on the re-jig PR to be merged:

https://github.com/orientechnologies/orientdb-docker/pull/53

lag-linaro commented 5 years ago

@luigidellaquila is this repo still being maintained?

I submitted PR #53 nearly a year ago. It has been reworked since too.

lag-linaro commented 5 years ago

It looks like new commits have been applied with conflicted with PR #53

I have rebased, fixed the conflicts and re-pushed the pull-request.

madrover commented 2 years ago

I'm sorry to bring this zombie issue back from the mists of time but, IMHO, since the new Apple laptops are starting to be more and more common this request becomes relevant again. I have just built an multiarch amd64/arm64 docker version of OrientDB using https://github.com/orientechnologies/orientdb-docker/blob/master/release/3.2.x/3.2.4/Dockerfile without any issue, so there does not seem to be much problem with it. However, it would be much nicer if we could just pull it from Dockerhub directly. Many thanks guys! :)

MarttiR commented 2 years ago

@luigidellaquila Would you be able to share any update on this issue?

Pulling orientdb (orientdb:3.2.4)...
3.2.4: Pulling from library/orientdb
ERROR: no matching manifest for linux/arm64/v8 in the manifest list entries
MarttiR commented 2 years ago

I was able to build an arm64 container from the official Dockerfile as suggested above, but this solution seems to have extremely poor performance – simply creating a database takes over 10 seconds.

semihalf-mazur-hubert commented 2 years ago

Hi, any updates about providing official docker images for arm64? Image build from official Dockerfile works fine.

tglman commented 2 years ago

Hi,

I saw some work has been done and merged, but one of the last PR was not merged, I will be happy to review and merge a PR that add arm64 support for 3.0.x, 3.1.x, 3.2.x without change other architectures.

Regards

semihalf-mazur-hubert commented 2 years ago

What is needed to add support for arm64? The Dockerfile is architecture independent and relies on eclipse-temurin which has the arm64 image.

matthewadams commented 1 year ago

We could use this as well. Getting tired of the "death by a thousand cuts" of using Apple Silicon. We have had to manually build & privately publish our own Docker images. :/

tglman commented 1 year ago

Hi,

ARM64 build should be published on docker.io