-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
storage
- add support for storage_account_id
to azurerm_storage_container
and azurerm_storage_share
#27733
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jackofallops.
Thinking ahead for when we make this breaking change in 5.0, would we be able to add a state migration for this so that users don't need to remove and re-import this into their state?
internal/services/storage/storage_container_data_source_test.go
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jackofallops LGTM 🐈⬛
This PR was generated via `$ upgrade-provider pulumi/pulumi-azure --kind=all --target-bridge-version=latest`. Manual edits: - [Added some new resources to resources.go that weren't automapped.](https://github.com/pulumi/pulumi-azure/pull/2632/files#diff-34c57e622183cb0d8dd0d3f9eaa0861b3340120e9b2ad811bac7ac7be4cea4b1) - [Changes to storage/zMixins.ts](https://github.com/pulumi/pulumi-azure/pull/2632/files#diff-2d6bc02db2d831a89f3ac5906d7bd812c1b1eb90afaeb9022bf39ebb2c537ea8) because [upstream introduced a change that makes storage account names optional](hashicorp/terraform-provider-azurerm#27733) (and deprecated). --- - Upgrading terraform-provider-azurerm from 4.8.0 to 4.9.0. Fixes #2610
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. |
Community Note
Description
This PR introduces the
storage_account_id
to supersede thestorage_account_name
property intoazurerm_storage_container
andazurerm_storage_share
. This allows the provider to create and manage these resources via the Resource Manager API rather than the Data Plane.Existing resources using the
storage_account_name
property cannot be migrated directly, but can be removed and re-imported using the Resource Manager ID type.The
storage_account_name
property is now deprecated in both resources and will be removed in v5.0 of the provider.The
resource_manager_id
property is now deprecated in both resources and will be removed in v5.0 of the provider, this value is available asid
in resources usingstorage_account_id
.PR Checklist
For example: “
resource_name_here
- description of change e.g. adding propertynew_property_name_here
”Changes to existing Resource / Data Source
Change Log
Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.
azurerm_storage_container
- support for thestorage_account_id
propertyazurerm_storage_share
- support for thestorage_account_id
propertyThis is a (please select all that apply):
Related Issue(s)
Closes #2977
NOTE: please see #27818 and #27819 for other issues that ended up being tracked/associated with the original issue.
Closes #27607
Note
If this PR changes meaningfully during the course of review please update the title and description as required.