2809 - Additional make commands required for Postgres upgrade#3511
Open
hopep-mobilise wants to merge 2 commits into
Open
2809 - Additional make commands required for Postgres upgrade#3511hopep-mobilise wants to merge 2 commits into
hopep-mobilise wants to merge 2 commits into
Conversation
RMcVelia
reviewed
Jun 3, 2026
hopep-mobilise
force-pushed
the
2809-postgres-upgrade-prereqs
branch
from
d832236 to
b9397c2
Compare
hopep-mobilise
force-pushed
the
2809-postgres-upgrade-prereqs
branch
from
b9397c2 to
0d16c8f
Compare
hopep-mobilise
force-pushed
the
2809-postgres-upgrade-prereqs
branch
from
0d16c8f to
d3c0c4a
Compare
Contributor
Author
|
@RMcVelia Martyn has successfully tested the review app and has requested an all clear review from yourself before we merge. |
RMcVelia
force-pushed
the
2809-postgres-upgrade-prereqs
branch
from
89dea78 to
317a3be
Compare
- Add AZURE_RESOURCE_PREFIX for Makefile set-pgserver - increase review app postgis to 17-3.6
RMcVelia
force-pushed
the
2809-postgres-upgrade-prereqs
branch
from
317a3be to
863600b
Compare
|
Review app deployed to https://get-school-experience-review-pr-3511.test.teacherservices.cloud |
Contributor
ok, originally it was still running 14 as it needs a different variable updated (fixed). All looks good infra wise now, but would be good if Martyn can recheck the review app attached to this PR as it's now using postgis at 17. |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Trello card
https://trello.com/c/6Ey6BDbj/2809-upgrade-gse-postgres-version
Context
We need to upgrade Postgres DB to version 17.
Changes proposed in this pull request
Additional commands have been added to Makefile to allow the ability to enable and disable Postgres logs and to scale Kubernetes pods.
The Postgres version has been parameterised with a default set to the current version. The Postgres version has been set to 17 for the review app.
Guidance to review
We need to test a review app on version 17 first. This PR will allow this to happen. The review app will then need to be tested by the service team.
The PR can be merged immediately as it will not affect any environment except review. Additional PRs will be required to upgrade the environments.