Skip to content

Conversation

@josielsouzanordcloud
Copy link
Contributor

@josielsouzanordcloud josielsouzanordcloud commented Dec 15, 2025

Description

This PR improves application resilience by adding configurable minimum replica counts for container apps.

Changes

  • Added min_replicas terraform variable to control minimum number of container replicas
    • Default value: 1
    • Preprod and Prod environments set to: 2 replicas
  • Added min_replicas parameter through terraform modules (terraform/main.tfcontainer-apps/main.tf → webapp container app)

Benefits

  • Improved availability: Preprod and Prod environments now run with 2 minimum replicas for better resilience
  • Configurable scaling: Each environment can now specify minimum replica count based on requirements

Jira link

DTOSS-11851

Review notes

Files Changed: infrastructure/modules/container-apps/front_door.tf - Added origin_group health_probe configuration

Review checklist

  • Check database queries are correctly scoped to current_provider

@github-actions
Copy link

github-actions bot commented Dec 15, 2025

The review app at this URL has been deleted:
https://pr-830.manage-breast-screening.non-live.screening.nhs.uk

- Add min_replicas variable to control minimum number of container replicas
- Set default min_replicas to 1, configure preprod and prod environments to 2 replicas
- Thread min_replicas parameter through terraform modules to container app
@josielsouzanordcloud josielsouzanordcloud force-pushed the DTOSS-11851-improve-app-resilience branch from fcdcd0a to 66042f8 Compare December 15, 2025 16:17
@josielsouzanordcloud josielsouzanordcloud changed the title [DTOSS-11851] - Improve App Resilience with Front Door Health Probes [DTOSS-11851] - Set minimum replicas to 2 for preprod and prod environments Dec 15, 2025
@josielsouzanordcloud josielsouzanordcloud merged commit 9c4a77c into main Dec 16, 2025
13 checks passed
@josielsouzanordcloud josielsouzanordcloud deleted the DTOSS-11851-improve-app-resilience branch December 16, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants