Skip to content

Update PostgreSQL minor to 17.3,16.7... via update.sh #408

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

Closed
wants to merge 1 commit into from

Conversation

thomasboussekey
Copy link
Contributor

Even if PostgreSQL development Group recommends not to use it, it may help some people that are exposed

Even if PostgreSQL development Group recommends not to use it, it may help some people that are exposed
@thomasboussekey
Copy link
Contributor Author

Tried to understand the CI error, but didn't succeed 😢

@ImreSamu
Copy link
Member

Thank you for the GitHub PR.

1.)
For future reference, if only a re-run is needed, please create a GitHub issue indicating that the Docker Postgres has been updated (meaning not only the GitHub repository but also the DockerHub repository is updated and the new Postgres Docker image is available). Then one of the maintainers can simply re-run the regular Monday job by clicking the "re-run jobs" button.
image

2.)
I already did this 13 hours ago, ( https://github.com/postgis/docker-postgis/actions/runs/13234206132 ~ "Re-run triggered 13 hours ago" ) and there was an issue with the two PostGIS master images - which were successful on Monday. This suggests that the new Postgres might have introduced some changes.

Apart from the master, it appears that all other PostGIS images have been updated.
( postgis/postgis:17-3.5 is now : 17.3 )

my simple check:

$ docker pull postgis/postgis:17-3.5
17-3.5: Pulling from postgis/postgis
Digest: sha256:0a5e5ed46da6883d5c615967f9fa185b1c3271cbe2392bac508233cb095f2c3d
Status: Image is up to date for postgis/postgis:17-3.5
docker.io/postgis/postgis:17-3.5

$ docker run --name some-postgis -e POSTGRES_PASSWORD=mysecretpassword -d postgis/postgis:17-3.5
0ef19cdef7f3f94bc53222fb8f277e76266b6a29bddda7aea88f4818d64148bc

$ docker exec -ti some-postgis psql -U postgres
psql (17.3 (Debian 17.3-1.pgdg110+1))
Type "help" for help.

postgres=# select version();
                                                           version                                                           
-----------------------------------------------------------------------------------------------------------------------------
 PostgreSQL 17.3 (Debian 17.3-1.pgdg110+1) on x86_64-pc-linux-gnu, compiled by gcc (Debian 10.2.1-6) 10.2.1 20210110, 64-bit
(1 row)

3.)
New Postgres release expected on :February 20, 2025 - expected to fix the 'master' issue.

4.)
Based on the build log, the minimal tests passed,
so end users will likely not notice any issues.
Nonetheless, I will keep an eye on the problem and, if necessary, we will revert to the previous Postgres version.

in the log:

docker images          postgis/postgis:17-3.5
REPOSITORY        TAG       IMAGE ID       CREATED         SIZE
postgis/postgis   17-3.5    648c9f0e2ee0   2 seconds ago   691MB
/home/runner/official-images/test/run.sh -c /home/runner/official-images/test/config.sh -c test/postgis-config.sh postgis/postgis:17-3.5
testing postgis/postgis:17-3.5
	'utc' [1/6]...passed
	'no-hard-coded-passwords' [2/6]...passed
	'override-cmd' [3/6]...passed
	'postgres-basics' [4/6]...passed
	'postgres-initdb' [5/6]...passed
	'postgis-basics' [6/6]...passed
..

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