Skip to content

Document Terraform v5 migration guidance - #1239

Merged
shibayan merged 1 commit into
masterfrom
agent/document-terraform-v5-migration
Aug 4, 2026
Merged

Document Terraform v5 migration guidance#1239
shibayan merged 1 commit into
masterfrom
agent/document-terraform-v5-migration

Conversation

@shibayan

@shibayan shibayan commented Aug 4, 2026

Copy link
Copy Markdown
Member

Summary

  • clarify that the in-place v4-to-v5 migration does not apply to the former Terraform module
  • recommend deploying a new Flex Consumption environment with the current v5 module
  • document how to reuse an existing Key Vault and which access settings must be verified

Why

The v4 and v5 Terraform modules have substantially different resource structures, so a Terraform state/import-based in-place migration is not supported. The migration guide should make the recommended re-setup path explicit.

Validation

  • npm run build in docs

Closes polymind-inc/terraform-azurerm-acmebot#35

@shibayan shibayan self-assigned this Aug 4, 2026
@shibayan shibayan added the documentation Improvements or additions to documentation label Aug 4, 2026
@shibayan
shibayan marked this pull request as ready for review August 4, 2026 13:38
Copilot AI review requested due to automatic review settings August 4, 2026 13:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the v4→v5 migration documentation to explicitly address Terraform-managed deployments, clarifying that the “in-place migration” guidance applies to an existing Function App deployment, not to the legacy Terraform module deployment model.

Changes:

  • Adds a “Terraform Module Deployments” section clarifying the legacy shibayan/keyvault-acmebot/azurerm module is not eligible for the in-place migration steps.
  • Documents the recommended approach for Terraform users: deploy a fresh v5 Flex Consumption environment using the polymind-inc/acmebot/azurerm module and retire the v4 environment.
  • Notes Key Vault reuse via acmebot.vault_uri and calls out the configuration areas that must be recreated/verified for the new deployment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@shibayan
shibayan merged commit fea7634 into master Aug 4, 2026
11 checks passed
@shibayan
shibayan deleted the agent/document-terraform-v5-migration branch August 4, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migration guide/advice for updating from old tf module (v4) to new (v5)

2 participants