Skip to content
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

Documentation should state which field changes will Force New #186

Open
displague opened this issue May 20, 2021 · 0 comments
Open

Documentation should state which field changes will Force New #186

displague opened this issue May 20, 2021 · 0 comments
Labels
area/resources/metal Issues related to Metal APIs source/terraform-provider-metal This issue/pr was originally created in deprecated terraform-provider-metal repository

Comments

@displague
Copy link
Member

Looking at the documentation, it is not clear if, for example, changing the metal_device user_data will cause the device to be recreated of if that user_data change will only update the metadata after the initial configuration. (relates to packethost/terraform-provider-packet#236)

Regardless of that situation, all ForceNew fields should include documentation that changes to this field will recreate the resource, with any special notes added as needed.

In the case of userdata, (I don't recall how it is treated), it might be expected for ForceNew behavior to exist. So we should also document cases where ForceNew may be expected, but this is not the case (and why).

@ocobles ocobles added the source/terraform-provider-metal This issue/pr was originally created in deprecated terraform-provider-metal repository label Jun 20, 2022
@ocobles ocobles transferred this issue from equinix/terraform-provider-metal Jun 20, 2022
@displague displague added the area/resources/metal Issues related to Metal APIs label Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/resources/metal Issues related to Metal APIs source/terraform-provider-metal This issue/pr was originally created in deprecated terraform-provider-metal repository
Projects
None yet
Development

No branches or pull requests

2 participants