-
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
New Resource: azurerm_disk_pool
#14675
Conversation
Hello @tombuildsstuff , should I continue my work #14268 based on this pr once it has been merged? It's your call, awaiting for your instruction. |
7dbdd4c
to
b2d2b04
Compare
Sku needs to be ForceNew in both of these:
|
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 @tombuildsstuff - LGTM 🏗️
This replaces the older `azurerm_storage_disk_pool` - which is completely unrelated to Storage. This is intentionally a replacement with some minor changes since `zones` should be consistent going forward
…we use the SDK version
The API also 500's if the name field is sent
35f38f3
to
d76aa12
Compare
This functionality has been released in v2.91.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! |
… Resource `azurerm_disk_pool` on pr #14675 (#14268) Co-authored-by: zjhe <[email protected]>
…ool` on pr #14675 (#14975) Co-authored-by: zjhe <[email protected]> Base on azurerm_disk_pool, added new resource azurerm_disk_pool_iscsi_target for disk pool. This resource HAVE NOT embedded lun arguments, I think it'll be better to make a new seperate resource to managed lun disk in an iscsi target, otherwise the user need delete embedded lun inside iscsi target resource before they can delete the managed disk and the attachment.
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. |
This PR introduces a new resource
azurerm_disk_pool
which supersedes the existingazurerm_storage_disks_pool
- this is because the existing resource is unrelated to the Storage package and is a collection of (Managed) Disks. Since there's a number of Disk related resources (and will be more with #14268) - this pulls the existing resource and this new resource out into it's own package.There's a follow up here to move Managed Disks into this package too, but that's outside the scope of this PR.
Dependent on #14673 and #14672