Skip to content
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

Pin postgis docker version #535

Merged
merged 1 commit into from
Nov 14, 2024
Merged

Pin postgis docker version #535

merged 1 commit into from
Nov 14, 2024

Conversation

floryst
Copy link
Contributor

@floryst floryst commented Nov 13, 2024

Existing deployment databases (postgresql v16) are not compatible with postgresql v17. Pin to v16 so we don't become incompatible with existing databases if the docker image gets deleted and re-downloaded.

@floryst floryst requested a review from mvandenburgh November 13, 2024 16:45
Copy link
Member

@mvandenburgh mvandenburgh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should also pin the scoringdb service.

Existing deployment databases (postgresql v16) are not compatible with
postgresql v17.
@floryst floryst force-pushed the pin-postgis-version branch from 5d82f9e to a60b6c0 Compare November 14, 2024 21:06
@floryst
Copy link
Contributor Author

floryst commented Nov 14, 2024

I force-pushed to include the scoringdb in the pinned version.

@floryst floryst requested a review from mvandenburgh November 14, 2024 21:07
@floryst floryst merged commit 7cf54ba into main Nov 14, 2024
8 checks passed
@floryst floryst deleted the pin-postgis-version branch November 14, 2024 21:24
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