equinix / terraform-provider-metal

DEPRECATED Equinix Metal standalone TF provider. Use the unified Equinix Terraform Provider.
https://registry.terraform.io/providers/equinix/equinix/latest/docs/guides/migration_guide_equinix_metal
Mozilla Public License 2.0
14 stars 11 forks source link

Device deletion should honor timeout settings #216

Closed displague closed 2 years ago

displague commented 2 years ago

The metal_device resource should support the Terraform timeout attributes.

Of particular interest is the timeout for delete events. During a reservation instance deletion the reservation state checking is hardcoded to 60m.

https://github.com/equinix/terraform-provider-metal/blob/main/metal/resource_metal_device.go#L785

The timeout attributes and defaults should be added and documented. The timeout should cover the time to delete the resource and the reservation cleanup.

enkelprifti98 commented 2 years ago

Just adding terraform docs for reference:

https://www.terraform.io/language/resources/syntax#operation-timeouts