New Resource: azurerm_management_group_subscription_association
#11069
Conversation
There was a problem hiding this comment.
Thanks @jackofallops - LGTM aside from one minor comment. Also does this superseded #8947 ?
| _, err = client.Create(ctx, id.ManagementGroup, id.SubscriptionId, "") | ||
| if err != nil { |
There was a problem hiding this comment.
these lines could be merged
That wasn't intentional, I didn't see that PR when I searched 🙈 , but yes this will replace that PR. |
jackofallops
force-pushed
the
r/subscription-mg-association
branch
from
55f6929
to
648d33a
Compare
…vider-azurerm * 'master' of github.com:terraform-providers/terraform-provider-azurerm: (68 commits) Update changelog for hashicorp#11164 `azurerm_cognitive_account` - Add support for `network_acls` (hashicorp#11164) Update CHANGELOG.md for hashicorp#11069 New Resource: `azurerm_management_group_subscription_association` (hashicorp#11069) tests: limit running VMware tests in teamcity (hashicorp#11167) CHANGELOG.md for hashicorp#10627 azurerm_firewall: support for private_ip_ranges (hashicorp#10627) Update CHANGELOG.md for hashicorp#11083 Upgrade hpc cache API version to 2021-03-01 (hashicorp#11083) CHANGELOG.md for hashicorp#10724 new resource: azurerm_media_live_event (hashicorp#10724) CHANGELOG.md for hashicorp#11108 upgrade go sdk to v52.6.0 (hashicorp#11108) CHANGELOG.md for hashicorp#11139 azurerm_api_management: make certificate_password optional (hashicorp#11139) CHANGELOG.md for hashicorp#10830 r/azurerm_data_factory_linked_service_azure_blob_storage: Fixing managed identity implementation (hashicorp#10830) CHANGELOG.md for hashicorp#11131 azurerm_storage_account - support for 'container_delete_retention_policy' (hashicorp#11131) CHANGELOG.md for hashicorp#11106 ...
|
This has been released in version 2.54.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example: provider "azurerm" {
version = "~> 2.54.0"
}
# ... other configuration ... |
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks! |
ghost
locked as resolved and limited conversation to collaborators
closes #10912
closes #3555
Breaking Change Notes:
The introduction of the
azurerm_management_group_subscription_associationcreates a breaking change in the use ofazurerm_management_group. It is now required to explicitly set an empty list to remove allsubscription_ids, where previously simply omitting this value would remove all Subscriptions from the Management group. Settingsubscription_idsonazurerm_management_groupis not supported when usingazurerm_management_group_subscription_association.