Repository navigation
Conversation
jonbamber
force-pushed
the
3087-environment-specific-postgres
branch
from
November 10, 2026 18:56
e157a3a to
fb1e964
Compare
jonbamber
marked this pull request as ready for review
November 11, 2026 15:56
There was a problem hiding this comment.
🔵 Needs a closer look
Human confirmation is required that the Test server’s manual PostgreSQL 16→18 upgrade is complete and the post-upgrade Terraform plan shows no changes, since live infrastructure state cannot be verified from this snapshot.
No open issues 🎉
What changed in this PR
Prepares the service’s Terraform configuration for PostgreSQL 18 by making the server version configurable. This PR must be merged only after the manual database upgrade.
Changes:
- Adds
postgres_server_versionwith a default of"18". - Passes the variable to the PostgreSQL module instead of hardcoding
"16".
| File | Description |
|---|---|
terraform/application/variables.tf |
Defines the PostgreSQL version variable, defaulting to 18. |
terraform/application/database.tf |
Uses the version variable in the PostgreSQL module. |
Review effort: Balanced
jonbamber
force-pushed
the
3087-environment-specific-postgres
branch
from
November 11, 2026 17:40
03bade9 to
520d03f
Compare
This branch has not been deployed
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
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.
Context
DO NOT MERGE UNTIL AFTER MANUAL DATABASE UPGRADE
This work is related to upgrading the PostgreSQL version for this service from version 16 to version 18.
https://trello.com/c/64RX275N/3087-upgrade-drd-postgres-version
This change updates the Test environment configuration to use PostgreSQL 18. There is no review app for this change.
The upgrade of the flexible server will be performed manually (due to
azurermTerraform provider limitations) and this PR should be merged after that manual upgrade.Changes proposed in this pull request
postgres_server_versionvariable interraform/application/variables.tf, defaulting to"18".terraform/application/database.tfpasses the variable to thepostgresmodule asserver_versioninstead of the hardcoded"16".Guidance to review
Before manual upgrade
make test terraform-plan DOCKER_IMAGE_TAG=41e68678726d036aa4e2dd8e2ea0fc5f9fd95908will show:module.postgres.azurerm_postgresql_flexible_server.main[0].version:16->18module.postgres.azurerm_postgresql_flexible_server_configuration.connection_throttling[0]will be destroyed (not supported in v18)After manual upgrade
make test terraform-plan DOCKER_IMAGE_TAG=600761fa0d15e571d6891c97740e578380478454should show no changes.Checklist