terraform-provider-azurerm
terraform-provider-azurerm copied to clipboard
`storage`: refactoring to use `hashicorp/go-azure-sdk`
This PR refactors the azurerm_storage_account Data Source and Resource and the azurerm_storage_account_network_rules Resource over to using hashicorp/go-azure-sdk.
This PR supersedes #23002 by @petr-stupka, since the base layer used in that PR is no longer used (and therefore the logic in it will no longer work) - and also supersedes #25789 by @wuxu92, since that’s only related to the Static Website endpoint. However this PR fixes #23240, #22992, #19055, #18897 and #15609 (which both were trying to fix in slightly different manners) by polling during the creation of the Storage Account, when applicable, for each type of supported sub-service (e.g. Blobs, Files, Static Websites etc) using a custom poller.
This also fixes #25160 and #12627 by switching to use hashicorp/go-azure-sdk’s base layer, which’ll correctly surface errors coming from the Azure API.
Fixes #25160 Fixes #23240 Fixes #22992 Fixes #22507 Fixes #19427 Fixes #19055 Fixes #18897 Fixes #15609 Fixes #14737 Fixes #12627 Fixes #26389
Changelog
(TBD)
@tombuildsstuff Any chance I can get you to look at this PR ? I've been hitting this consistently while creating new storage accounts and I can't figure if its my setup or azure or the provider that is the problem.
Any update on the timeline for this fix?
Test results:
CDN
Compute
failures are 4.0 related
Storage
remaining failures are 4.0 related
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.