fix: Update Fabric Terraform Resources and Data sources for latest Fabric …

equinix / terraform-provider-equinix

Terraform Equinix provider

https://deploy.equinix.com/labs/terraform-provider-equinix/

MIT License

46 stars 45 forks source link

fix: Update Fabric Terraform Resources and Data sources for latest Fabric … #799

Closed thogarty closed 4 weeks ago

thogarty commented 1 month ago

Upgrade equinix-sdk-go to v0.46.0
Update Port models to match changes in new Fabric SDK
Update SimplifiedAccount to match changes in new Fabric SDK

ctreatma commented 1 month ago

Does this need to be synchronized with API deployment in any way?

Does this need to be released immediately because customers using existing versions of the TF provider are broken due to API changes?
Does this need to remain unreleased until after the API changes roll out (if 4.17 hasn't been released yet)?

thogarty commented 1 month ago

Closes #798

thogarty commented 1 month ago

@ctreatma this should be released immediately to give customers a version that they can upgrade to once 4.17 API changes rollout. We do expect that early upgrades will break, but we would rather have that then not have a version available to customers that they can upgrade to for fixing the breaking issues.

The 4.17 API release is tomorrow evening (10/18)

thogarty commented 4 weeks ago

The release notes will need to include a note that all configurations with equinix_fabric_port and equinix_fabric_ports will need to updated to the latest version.

Migration notes will need to be included if the state will not be safely updated in-place.

Agree. The state should be able to be updated with a refresh with no other changes than an upgrade. Just modifying internal functionality and not the Terraform interface. Additionally, for ports we have no resources.

Merging this in.

github-actions[bot] commented 4 weeks ago

This PR is included in version 2.9.0 :tada: