fix: Pin embedded postgresql version to 16 #770
Merged
+7
−4
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.
In #753 I upgraded from node:20-alpine3.19 to node:24-alpine3.23. In alpine 3.23, I inadvertently introduced two version changes to our internally packaged redis & postgres dependencies:
redisupgraded from 7.2.9 to 8.4.0postgresqlupgraded from 16 to 18.Redis can safely upgrade from 7->8 without issue. However, upgrading from postgres 16->18 (or between any postgres major versions for that matter) will result in errors that look like the following:
This PR pins the postgresql version we ship in the embedded Docker container to 16. Additionally, it pins the postgres and redis versions used in the docker-compose.yml to 16 & 8 respectively.
Note: By pining versions in the docker-compose, there may be instances where folks run into errors during migration when 1) they updated their
docker-compose.ymlto this new revision with pinned versions, and 2) their previouslatestfor postgres was > 16. In this case, thePOSTGRES_VERSIONenv-var can be set to whatever version they were on before.Fixes #768