-
Notifications
You must be signed in to change notification settings - Fork 469
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 arm64 support using buildx #256
base: master
Are you sure you want to change the base?
Conversation
7ebe031
to
78f44ff
Compare
@phillipross, |
@odidev : A few hours ago .. the upstream
|
I think there's going to be more work needed to get everything working well here with both platforms.
I see that @odidev is actively pushing changes to the PR, but as it stands now, the PR only builds the multiplatform images as part of the push target which is problematic since the push target is only invoked when PRs are merged to the main repo. This needs to change so that the build targets build the images, the test targets run the tests against the images that are built (this is a challenge, i know), and then the push target pushes the blessed images to the image repo. I'm not sure if this PR is the best place to discuss all of this or not. Maybe we should use the existing issue #216 to discuss? |
Signed-off-by: odidev <[email protected]>
Hi there, is there any update on this PR? |
Hi, would it also be possible to add additional arm32 (linux/arm/v7) support? |
@RyanSiu1995 No update yet, we're still working on it
@Tob1as I'm sure it will be possible, but with some work. |
Hi, any update on this MR pls? 🙏 |
Any update on this? |
What remains to be done to add arm64 (linux/arm64/v8) as an architecture in the official postgis DockerHub images? I'm happy to help in any capacity. I see a huge performance loss running the AMD64 images on the new Apple M series Macs. The official Postgres images have added linux/arm64/v8 to all supported images. Given the popularity of this platform I think it makes sense to support it as an official architecture. |
I am currently resorting to copying the Dockerfile and scripts into our development environment so that my co-workers using Apple laptops with M1/2 chips are able to run Postgres + PostGIS by building it themselves. I just revisited that code because PostGIS 3.3.2 appears to have been removed from Apt and the build broke - it took me quite a few minutes to understand why I would ever have copied someones Dockerfile so closely. I don't understand why there is no ARM support when it is fully supported by the base image/dependencies in the Dockerfile; these architectures have gained a ton of popularity in the past 5 years- especially in cloud computing where PostGIS is needed. |
Compounding matters is that the docker-postgis team doesn't have any real form of release management. They have a single branch that they keep up to the latest version, and that's it. So for those of us on arm64:
We're stuck with having to use the latest version. We'd prefer to match our Google Cloud SQL environment, which supports only up to 3.1.8, not 3.3.3. |
Hello.....? I'm stumped. I'm not sure what else to say. Bump, please! |
@nitrag : Work in progress ...
status: testing .... |
The following file has been created and modified:
Added support of buildx in Makefile to build and push the docker image for arm64 and amd64 platforms.
Updated .github/workflows/main.yml to build, test and push the images for the latest versions to docker hub.
Signed-off-by: odidev [email protected]