July 2024 update #346
Merged
July 2024 update #346
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
g7morris
added
fits
activemq
handle
tomcat
fcrepo
documentation
nigelgbanks
requested changes
Aug 1, 2024
| @@ -1,6 +1,6 @@ | |||
| # PostgreSQL | |||
|
|
|||
| Docker image for [PostgreSQL] version 15.1 | |||
| Docker image for [PostgreSQL] version 16.3 | |||
There was a problem hiding this comment.
I got 16.2 when I checked.
/ $ postgres --version
postgres (PostgreSQL) 16.2
There was a problem hiding this comment.
Hmmmm I'm not getting the same results from a fresh build?
- Reran
make clean && make bake && make up
Check 1:
docker run --rm -d --name postgresql islandora/postgresql
docker exec -ti postgresql psql -U root postgres
psql (16.3)
Type "help" for help.
postgres=#Check 2:
docker exec -ti postgresql bash
6f89c38c08af:/# postgres --version
postgres (PostgreSQL) 16.3There was a problem hiding this comment.
My local build matches the version from the latest release:
make bake BUILDER=desktop-linux TARGET=postgresql 2&>/dev/null && docker run --rm -ti -u 1 --entrypoint psql islandora/postgresql:local --version
psql (PostgreSQL) 16.2
So, I obliterated all my local builds and then built without access to the cache on DockerHub and got what you have. Which is worrying, I'll open up a ticket to investigate this cache poisoning issue.
nigelgbanks
approved these changes
Aug 2, 2024
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Release notes:
All services below have an updated README.md, new software at the latest version, new SHA etc. along with having been tested via processes outlined here if there should be any questions / concerns: https://gist.github.com/g7morris/05818f8da4bac3c129063f1478f9b078
activemqfitsfits-servlet- updated softwarelog4j- updated softwarehandletomcatUpdated service README.mds to latest version only.
fcrepofitsmariadbnginxpostgresqlsolr@nigelgbanks and @joecorall Once reviewed and approved, either one or both of you can merge in but if there are too many commits, I'm happy to squash them down to one if need be. Let me know please and thanks in advance!