-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upgrade to postgres 10.15 #148
Conversation
Run the functional tests locally. Smoke tests all OK. Some of the full suite failed - now to see whether that's a pre-existing condition... |
Yes - the same tests failed on Postgres 9 and 10 because I didn't have a Notify key. I think this is good to go. |
ports: | ||
- "5432:5432" | ||
volumes: | ||
- ./sql:/docker-entrypoint-initdb.d | ||
- postgres-data:/var/lib/postgresql/data | ||
environment: | ||
POSTGRES_USER: $USER | ||
POSTGRES_HOST_AUTH_METHOD: trust |
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.
Why is this needed?
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.
Argh - meant to break this out into a separate commit, but forgot. It's due to a change in the Docker container that now requires us to be explicit that we don't want a root password: docker-library/postgres#658
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.
I've amended the commit description to include this. Will now merge when green
We need to upgrade to Postgres 12 by mid-February. We can't go directly, we need to go via Postgres 10 - https://docs.cloud.service.gov.uk/deploying_services/postgresql/#upgrade-to-postgresql-10. Upgrade dmrunner to use postgres 10 so we can test locally. After pulling this commit, you will need to run `make data` to re-initialise the database. We need to add `POSTGRES_HOST_AUTH_METHOD` because the postgres docker container has changed to require it: docker-library/postgres#658
Trello: https://trello.com/c/WdULXylT/2141-upgrade-postgresql
We need to upgrade to Postgres 12 by mid-February. We can't go directly, we need to go via Postgres 10 - https://docs.cloud.service.gov.uk/deploying_services/postgresql/#upgrade-to-postgresql-10.
Upgrade dmrunner to use postgres 10 so we can test locally. After pulling this commit, you will need to run
make data
to re-initialise the database.