orientechnologies / orientdb-docker

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

Adding dockerfile changes for ppc64le #59

Open ghatwala opened 5 years ago

ghatwala commented 5 years ago

My intent is to have "ppc64le" built officially - https://github.com/docker-library/official-images/blob/master/library/orientdb For Dockerfile which contain "openjdk:8-jdk-slim" as the base image wont work for power as tag due to below reason "no matching manifest for linux/ppc64le in the manifest list entries" As "alpine" base image is already multi-arch image , those images get created successfully. Have created "ppc64le" folders for all versions as done for "x86_64".

Confirmation of the image builds completing successfully on ppc64le m/c for the versions sought to be added in this PR.

# arch
ppc64le

# docker images | grep "2.0.18"
orientdb                                               2.0.18                         8c243f58e671        5 hours ago         717MB
# docker images | grep "2.2.37"
orientdb                                               2.2.37                         bbdeceaa4974        5 hours ago         171MB
# docker images | grep 3.0.21
orientdb                                               3.0.21                         23aca82658e3        3 hours ago         161MB

@luigidellaquila , @laa - Any review comments here - please ?

ghatwala commented 5 years ago

@luigidellaquila - Any possibility of this PR review ... we can add power support to releases and help orientdb docker image officially supported on power.

luigidellaquila commented 5 years ago

Hi @ghatwala

The problem here is that alpine image is no longer supported by DockerHub (see https://github.com/docker-library/official-images/pull/6018 and https://github.com/docker-library/openjdk/pull/322) so I don't know if this PR makes much sense in general...

Thanks

Luigi

ghatwala commented 5 years ago

Hey @luigidellaquila , thanks for the quick repsonse here ...

I agree on the alpine not being supported officially for any orientdb version - https://github.com/docker-library/official-images/blob/master/library/orientdb . Is there any other way to add power support to the supported version of orientdb - Any pointers ??

Any thoughts/comments on above ?