-
Notifications
You must be signed in to change notification settings - Fork 468
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
experimental buildx.yml #312
base: master
Are you sure you want to change the base?
Conversation
12d0396
to
cca3f1b
Compare
imho review ready |
A lot of changes have been made( Postgis 3.3; Postgres 15 Beta4 ) |
I agree that a separate tagging scheme should be used for images produced by this alternate workflow for the time being. It looks like there's still a bunch of things to figure out, but this is a great start! If it's possible to still have CI runs occur for PRs in draft mode, maybe you can switch this PR to draft mode so that it doesn't get accidentally merged? |
PR converted to "draft"
ok . . I will add an experimental postfix .. ( example |
583ebba
to
e961c2d
Compare
e961c2d
to
a7e322a
Compare
A lot of improvements / changes :
important: the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great work so far! I added some comments for potential changes to reduce some hardcoding, but I'm also testing all of this in a VM in my local environment to see how portable it is.
Added some new ENV vars + other small improvements; example;
Ready to review ... |
a2a447f
to
2c95d9d
Compare
PROPOSAL ( updated at 2022-10-16 )
new workflow
github/workflows/buildx.yml
docker buildx
( qemu emulation ) for the non-AMD64 platformsgithub/workflows/main.yml
workflow.The proposed
buildx.yml
matrix :Experimental linux/arm64 images
-experimental
tag postfix!linux/arm64
postgis/postgis:10-3.2-experimental
linux/arm64
postgis/postgis:11-3.3-experimental
linux/arm64
postgis/postgis:12-3.3-experimental
linux/arm64
postgis/postgis:13-3.3-experimental
linux/arm64
postgis/postgis:14-3.3-experimental
linux/arm64
postgis/postgis:15-3.3-experimental
linux/arm64
postgis/postgis:15-3.3-alpine-experimental
Other Experimental images
-experimental
tag postfix!linux/arm/v7
postgis/postgis:15-3.3-alpine-experimental
linux/arm/v6
postgis/postgis:15-3.3-alpine-experimental
linux/386
postgis/postgis:15-3.3-alpine-experimental
linux/ppc64le
postgis/postgis:15-3.3-alpine-experimental
linux/ppc64le
postgis/postgis:15-3.3-experimental
minimal post-tests:
postgres-basics
- https://github.com/docker-library/official-images/tree/master/test/tests/postgres-basics$POSTGRES_TEST_TRIES
;$POSTGRES_TEST_SLEEP
postgres-initdb
- https://github.com/docker-library/official-images/tree/master/test/tests/postgres-initdb$POSTGRES_TEST_TRIES
;$POSTGRES_TEST_SLEEP
postgis-basics
- https://github.com/postgis/docker-postgis/tree/master/test/tests/postgis-basics ( patched )$POSTGRES_TEST_TRIES
;$POSTGRES_TEST_SLEEP
The default $POSTGRES_TEST_TRIES; $POSTGRES_TEST_SLEEP is not enough for the emulation
so I added a higher value for the
buildx.yml
Lesson learned / improvements:
LOG: server process (PID 28990) was terminated by signal 7: Bus error
JIT=OFF
solved the problems in the alpine regression test. ( so added to workaround to the alpine.template )make test
DOCKER_DEFAULT_PLATFORM=
,PLATFORM=
environment variables!github/workflows/buildx.yml
uname -a
;uname -m
;lscpu
;cat /proc/cpuinfo
- useful for the buildx/qemu debugging.NOW: using different tagging for the "buildx build" !
-experimental
tag postfix has been added to the image names in the buildx.yml!15-3.3-experimental
/15-3.3-alpine-experimental
NEW environment variables: ( Makefile, workflow )
WORKFLOW
: main,buildx ( workflow id )TAG_POSTFIX
: <>,'-experimental' ( Docker image tag postfix )IMPORTANT:
Makefile
syntax; so please double-check!docker push
not tested!