azurerm_postgresql_flexible_server: force new when downgrading storage_mb

[wuxu92]

Community Note

• Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
• Please do not leave comments along the lines of "+1", "me too" or "any updates", they generate extra noise for PR followers and do not help prioritize for review

Description

We should tag the resource force new rather than raising an error. fixes #28942

related links: #25986

Testing

• My submission includes Test coverage as described in the Contribution Guide and the tests pass. (if this is not possible for any reason, please include details of why you did or could not add test coverage)

Terraform will perform the following actions:

  # azurerm_postgresql_flexible_server.test must be replaced
-/+ resource "azurerm_postgresql_flexible_server" "test" {
      - administrator_password_wo_version = 0 -> null
      ~ backup_retention_days             = 7 -> (known after apply)
      ~ fqdn                              = "xuwu1-pg1.postgres.database.azure.com" -> (known after apply)
      ~ id                                = "/subscriptions/xxx/resourceGroups/RG-xuwu1-Case/providers/Microsoft.DBforPostgreSQL/flexibleServers/xuwu1-pg1" -> (known after apply)
      ~ name                              = "xuwu1-pg1" -> "xuwu1-pg2" # forces replacement
      + private_dns_zone_id               = (known after apply)
      ~ storage_mb                        = 131072 -> 65536 # forces replacement
      ~ storage_tier                      = "P10" -> (known after apply)
      - tags                              = {} -> null
        # (13 unchanged attributes hidden)

      ~ authentication (known after apply)
      - authentication {
          - active_directory_auth_enabled = false -> null
          - password_auth_enabled         = true -> null
            # (1 unchanged attribute hidden)
        }
    }

Plan: 1 to add, 0 to change, 1 to destroy.
....
azurerm_postgresql_flexible_server.test: Still creating... [4m0s elapsed]
azurerm_postgresql_flexible_server.test: Creation complete after 4m8s [id=/subscriptions/xxx/resourceGroups/RG-xuwu1-Case/providers/Microsoft.DBforPostgreSQL/flexibleServers/xuwu1-pg2]

Apply complete! Resources: 1 added, 0 changed, 1 destroyed.

Change Log

Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.

• azurerm_postgresql_flexible_server - force new when downgrading storage_mb property [azurerm_postgresql_flexible_server validation for "storage_mb" value parameter #28942]

This is a (please select all that apply):

• Bug Fix
• New Feature (ie adding a service, resource, or data source)
• Enhancement
• Breaking Change

Related Issue(s)

Fixes #28942

Note

If this PR changes meaningfully during the course of review please update the title and description as required.

[catriona-m]

Thanks @wuxu92 LGTM!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.