Skip to content

Remove invalid Terraform moved block for sa_import_external#4993

Closed
rudolphjacksonm with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-code-for-review-comment-again
Closed

Remove invalid Terraform moved block for sa_import_external#4993
rudolphjacksonm with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-code-for-review-comment-again

Conversation

Copilot AI commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

What is being addressed

A review comment identified an invalid Terraform moved block in core/terraform/airlock/storage_accounts.tf: it attempted to move azurerm_storage_account.sa_import_external while that source address is still declared in configuration. This can cause terraform plan/apply to fail with moved-block configuration errors.

How is this addressed

  • Scoped fix for the referenced review comment

    • Removed only the invalid moved block mapping:
      • from = azurerm_storage_account.sa_import_external
      • to = azurerm_storage_account.sa_airlock_core
  • No broader refactor

    • Kept all other storage-account resources and move mappings unchanged to stay strictly within the single-comment scope.
  • Resulting top-of-file shape

    moved {
      from = azurerm_storage_account.sa_import_in_progress
      to   = azurerm_storage_account.sa_airlock_workspace_global
    }
  • Docs / changelog / versioning

    • No documentation updates required for this targeted Terraform metadata correction.
    • No CHANGELOG.md or template version updates required for this scope.

Copilot AI requested review from Copilot and removed request for Copilot July 23, 2026 09:51
Copilot AI changed the title [WIP] Fix code based on review comment Remove invalid Terraform moved block for sa_import_external Jul 23, 2026
Copilot AI requested a review from rudolphjacksonm July 23, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants