[jonbamber]

Context

This change is to remove the Redis resources as they are not used.

https://trello.com/c/GmwGMitl/3089-service-drd-migrate-redis

Changes proposed in this pull request

This change completely removes Redis configuration (the redis-cache module block in terraform/application/database.tf) - the effect of this will be to destroy the Cache for Redis resources.

The application does not reference Redis: there are no Redis environment variables, secrets or configuration in the application, and no Redis gem in the Gemfile. When checked, command statistics from the Redis cache showed only platform and polling commands, with no application commands.

Guidance to review

Run the following command to generate a Terraform plan and ensure planned changes are as expected:

• make test terraform-plan DOCKER_IMAGE_TAG=e16bef8b0761fdb9bed00c445745b781b5f02132 will show Plan: 0 to add, 0 to change, 2 to destroy with the removal of Redis resources:
  • module.redis-cache.azurerm_redis_cache.main[0]
  • module.redis-cache.azurerm_private_endpoint.main[0]

Checklist

• Tested by running locally
• Performed self-review
• Rebased onto main
• Cleaned commit history
• Attached to Trello card

[Copilot]

🔵 Needs a closer look

This change destroys live infrastructure, and unavailable Terraform state and plans require human verification that each environment’s plan removes only the unused Redis cache and private endpoint before applying.

No open issues 🎉

What changed in this PR

Removes the unused Redis Terraform module so the cache infrastructure can be decommissioned without changing the PostgreSQL configuration.

Changes:

• Deletes the redis-cache module and its configuration.
• Intended to destroy the Azure Redis cache and its private endpoint on the next Terraform apply.

File	Description
terraform/​application/​database.tf	Removes the Redis module while leaving PostgreSQL unchanged.

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

The change destroys deployed infrastructure, and the Terraform plan could not be independently validated to confirm that only the unused Redis cache and private endpoint are removed.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

The environment-specific Terraform plan needs independent verification that destruction is limited to the intended Redis cache and private endpoint, as live state and Redis usage telemetry were unavailable.

No open issues 🎉

_{Review effort: Balanced}

[jsomerset]

Redis resources removed in plan and no other changes