-
Notifications
You must be signed in to change notification settings - Fork 47
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
Timeouts are not honored on Metal resources/data sources #357
Comments
ctreatma
pushed a commit
that referenced
this issue
Aug 30, 2023
Provides support for timeouts in Create, Update and Delete for Metal Device as already mentioned in the documents #357
This was referenced Sep 13, 2023
ctreatma
added a commit
that referenced
this issue
Sep 21, 2023
ctreatma
added a commit
that referenced
this issue
Sep 22, 2023
ctreatma
added a commit
that referenced
this issue
Sep 25, 2023
As part of fixing the timeouts for metal resources to actually comply with what is documented, it was found out that we donot document any timeout support for metal VRF whereas in the code we do have support for timeouts. Since it has never been working due to the below issue, also not documented, it'll be ideal to remove it directly considering a code smell Relates to: #357
ctreatma
pushed a commit
that referenced
this issue
Sep 26, 2023
This PR will fix the timeout support for metal spot market requests which is already documented publicly relates to: #357
ctreatma
pushed a commit
that referenced
this issue
Sep 27, 2023
- Adds support for timeout in Create, update and delete operation. - Adds acceptance tests Fixes #357 --------- Signed-off-by: Ayush Rangwala <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Community Note
Terraform Version
v1.5.4
Affected Resource(s)
Terraform Configuration Files
References
Some of the Metal resources/data sources (for example, the equinix_metal_device resource) indicate support for timeouts in their docs and schema, but do not actually support those timeouts.
Per terraform-plugin-sdk v2 docs, we should be using CreateContext, DeleteContext, etc. functions in order to have terraform automatically handle timeouts on our behalf: https://developer.hashicorp.com/terraform/plugin/sdkv2/guides/v2-upgrade-guide#more-support-for-context-context
Also, for the resources and datasources when the
context aware
timeouts would not be suitable, use Hashicorp's WaitForStateContextWe should find any Metal resources & data sources that have a Timeouts schema property and ensure that those resources & data sources use context-aware CRUD functions.
Tasks
Upgrade resources and data sources to support context aware or without context CRUD timeouts
The text was updated successfully, but these errors were encountered: