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

packngo is deprecated. Use equinix-sdk-go #402

Open
14 of 41 tasks
Tracked by #99
displague opened this issue Oct 4, 2023 · 0 comments
Open
14 of 41 tasks
Tracked by #99

packngo is deprecated. Use equinix-sdk-go #402

displague opened this issue Oct 4, 2023 · 0 comments
Labels
area/resources/metal Issues related to Metal APIs

Comments

@displague
Copy link
Member

displague commented Oct 4, 2023

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
  • Please do not leave +1 or me too comments, they generate extra noise for issue followers and do not help prioritize the request.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Terraform Version

Affected Resource(s)

Resources

Data Sources

Terraform Configuration Files

# Copy-paste your Terraform configurations here.
#
# For large Terraform configs, please use a service like Dropbox and share a link to the ZIP file.
#
# If reproducing the bug involves modifying the config file (e.g., apply a config,
# change a value, apply the config again, see the bug), then please include both:
# * the version of the config before the change, and
# * the version of the config after the change.

Debug Output

Panic Output

Expected Behavior

Actual Behavior

Steps to Reproduce

  1. terraform apply

References

  • #0000
@displague displague added the area/resources/metal Issues related to Metal APIs label Nov 14, 2023
@ctreatma ctreatma changed the title packngo is being deprecated. Use metal-go packngo is being deprecated. Use equinix-sdk-go Jan 18, 2024
ctreatma added a commit that referenced this issue Jan 19, 2024
)

This updates the Equinix Metal project resource and data source from the
deprecated `packngo` SDK to `equinix-sdk-go`.

In order to change which SDK is used by the project resource, I also
needed to [rewrite some tests so that they use a mock API instead of a
mock
SDK](dc88ae0).

Relates to #402
ctreatma pushed a commit that referenced this issue Feb 8, 2024
#557)

This updates the Equinix Metal project resource and data source from the
deprecated packngo SDK to equinix-sdk-go and moves them into separate
package required for supporting plugin framework

Relates to
#402

---------

Signed-off-by: Ayush Rangwala <ayush.rangwala@gmail.com>
ctreatma added a commit that referenced this issue Feb 13, 2024
#402

Co-authored-by: codinja1188 <3358152+vasubabu@users.noreply.github.com>
Co-authored-by: Charles Treatman <ctreatman@equinix.com>
@ctreatma ctreatma changed the title packngo is being deprecated. Use equinix-sdk-go packngo is deprecated. Use equinix-sdk-go Jun 19, 2024
displague added a commit that referenced this issue Jun 20, 2024
…#700)

This updates the `metal_virtual_circuit` resource and data source to use
`equinix-sdk-go` instead of `packngo`. As mentioned in #402, the
`packngo` SDK has been deprecated, and moving to `equinix-sdk-go` makes
it possible for these resources to gain new features in the future.

As an added bonus, the affected code has also been moved to
`internal/resources/metal` to align with #106.
ctreatma added a commit that referenced this issue Jul 8, 2024
This replaces `packngo` with `equinix-sdk-go` in the metal_port resource
and datasource as part of #402. While I'm in these files I also opted to
move them into `internal/resources/metal/port` to align with #106.
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
Projects
None yet
Development

No branches or pull requests

1 participant