Add managed_resources block to purview_account 14814

[Noel-Jones]

Supports the managed_resource block requested in my enhancement resquest issue #14814

This is my first Terraform change therefore grateful for consideration in respect of any missed steps but hopefully I have covered it all!

With respect to the attribute names, I have kept them in sync with the Rest API - I considered it better to be in sync than to add "_id" to the attributes, however, I will do this if this is considered best practice for resource IDs. The documentation states that they are IDs.

Tested within our infrastructure, plan results in the following output against an existing resource and equivalent for a new resource.

Terraform detected the following changes made outside of Terraform since the last "terraform apply":

  # azurerm_purview_account.purview_account has changed
  ~ resource "azurerm_purview_account" "purview_account" {
        id                                               = "/subscriptions/00000-0-0-00000/resourceGroups/rg-noel-jones/providers/Microsoft.Purview/accounts/purview-sandbox"
      + managed_resources                                = [
          + {
              + event_hub_namespace = "/subscriptions/00000-0-0-00000/resourceGroups/rg-purview-sandbox/providers/Microsoft.EventHub/namespaces/Atlas-00000-0-0-00000"
              + resource_group      = "/subscriptions/00000-0-0-00000/resourceGroups/rg-purview-sandbox"
              + storage_account     = "/subscriptions/00000-0-0-00000/resourceGroups/rg-purview-sandbox/providers/Microsoft.Storage/storageAccounts/scanuksouthhjswedh"
            },
        ]
        name                                             = "purview-sandbox"
        tags                                             = {}
        # (10 unchanged attributes hidden)
    }

Signed-off-by: Noel Jones [email protected]

[katbyte]

Thanks @Noel-Jones - LGTM 🏗️

[tombuildsstuff]

hey @Noel-Jones

Thanks for this PR - taking a look through here since these are all Resource IDs can we add an _id suffix to each of these fields, so that it's clear these are Resource IDs rather than the names?

Thanks!

[Noel-Jones]

Thanks for the reviews and suggestion. I have committed the suggestions and retested. All looks good.

[katbyte]

Thanks @Noel-Jones - LGTM take 2 🚀🚀

[github-actions]

This functionality has been released in v2.92.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

[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.