Skip to content

feat: [PAYMCLOUD-574] Update Terraform azurerm module and PDF engine configuration#3618

Open
ffppa wants to merge 11 commits intomainfrom
PAYMCLOUD-574-AppService-itn-private
Open

feat: [PAYMCLOUD-574] Update Terraform azurerm module and PDF engine configuration#3618
ffppa wants to merge 11 commits intomainfrom
PAYMCLOUD-574-AppService-itn-private

Conversation

@ffppa
Copy link
Contributor

@ffppa ffppa commented Jan 29, 2026

List of changes

  • Updated PDF engine configurations:
    • Removed registry references from docker_image definitions.
    • Adjusted private DNS zone references for compatibility with Azure Web Apps private endpoints.
    • Added allowed_service_tags for improved network configuration.
    • Removed staging slot configurations for PDF engine modules.
  • Upgraded Terraform azurerm module to v8.3.5.
  • Migrated and refactored Terraform azurerm provider resources from v3 to v4.
  • Aligned configurations with the v4 module structure, improving maintainability and feature parity.

Motivation and context

This change ensures compatibility with the latest Terraform azurerm module and provider versions. It optimizes PDF engine configurations for better network connectivity and removes deprecated elements, improving overall maintainability and performance.

Type of changes

  • Add new resources
  • Update configuration to existing resources
  • Remove existing resources

Does this introduce a change to production resources with possible user impact?

  • Yes, users may be impacted applying this change
  • No

Does this introduce an unwanted change on infrastructure? Check terraform plan execution result

  • Yes
  • No

Other information

This update also includes a refactor to align with the updated Terraform provider structure, streamlining resource and module management for future changes.


If PR is partially applied, why? (reserved to mantainers)

ffppa added 2 commits January 27, 2026 18:56
…3650)

Upgrade Terraform azurerm provider to v4 and migrate all related modules and resources from v3 to v4. Add new variables and settings to align with the v4 module structure. Refactor configurations to enhance maintainability and enable feature parity with the updated provider version.

Signed-off-by: ffppa <fabio.felici@pagopa.it>
…odule

- Refactor `docker_image` definitions for PDF engines to remove registry references.
- Update private DNS zone references to align with Azure Web Apps private endpoints.
- Add `allowed_service_tags` for improved network configuration.
- Remove staging slot configurations for both PDF engine modules.
- Upgrade Terraform azurerm module to v8.3.5.

Signed-off-by: ffppa <fabio.felici@pagopa.it>
@ffppa ffppa requested review from a team as code owners January 29, 2026 16:09
@ffppa ffppa added env: dev The status refers to DEV environment status: partially applied labels Jan 29, 2026
@ffppa ffppa changed the title refactor: [PAYMCLOUD-574] Update Terraform azurerm module and PDF engine configuration feat: [PAYMCLOUD-574] Update Terraform azurerm module and PDF engine configuration Jan 30, 2026
ffppa added 5 commits January 30, 2026 11:53
- Migrate app service and staging slot references to updated module paths.
- Add `idh_app_service_resource_tier` variable with "premium" tier in UAT and PROD environments.

Signed-off-by: ffppa <fabio.felici@pagopa.it>
…urerm module

- Remove redundant autoscale resource definitions, consolidating settings within `autoscale_settings` blocks.
- Update maximum capacity and thresholds for requests, response time, and CPU usage for improved scaling behavior.
- Upgrade azurerm Terraform module to v8.5.3 for additional compatibility and features.

Signed-off-by: ffppa <fabio.felici@pagopa.it>
@ffppa ffppa added the env: uat The status refers to UAT environment label Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

env: dev The status refers to DEV environment env: uat The status refers to UAT environment status: partially applied

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant