equinix / equinix-sdk-go Goto Github PK

The FindIPReservations endpoint only has an Execute() method even though the endpoint is paginated.

The ExecuteWithPagination method is missing because the request object does not have a page parameter, which means there's no way to tell this generated client to fetch pages beyond 1.

The API spec for the findIPReservations path needs to be updated to reference the page parameter. There is an example of referencing the page parameter in the findProjectEvents endpoint.

As of now, this repo is set to private/internal while we fill in some gaps to make it ready for other teams & for external customers. Once this is ready for public consumption, we will delete the repo and recreate it to ensure that any unnecessary releases/tags/etc. are cleaned up.

Before this goes public, we need to:

• Fill in the README with customer-facing information
• Add a CONTRIBUTING document that includes contribution guidelines
• Add a DEVELOPMENT document that includes: how to get set up for local development, how to onboard a new service, how to update an existing service, how to cut a release
• Update examples to use this SDK instead of metal-go to demonstrate that this is a drop-in replacement
• Enable branch protection on main

Steps for going public:

• Cut a release that only includes the module name / package structure change (is otherwise identical to the last metal-go release
• Transfer remaining issues from equinix-labs/metal-go to this repo
• Deprecate equinix-labs/metal-go and include a link to this repo
• Update or replace metal-go Labs page with a page for this SDK

Currently when a service is onboarded to this SDK, we create a service-specific workflow that regenerates the code for that service in order to validate changes in Pull Requests. However, since this is a single SDK we need to ensure that there are no unexpected interactions with other services.

Remove the service-specific codegen workflows and replace them with a single workflow that runs make generate-all to regenerate the code for all onboarded services.

This could prevent cancelation

All ASN fields (peer_as, origin_as (below), peer_asn, local_asn, asn, bgp_neighbor_asn) should be updated to int64 with a min:0 and max:2^32 (since there is no uint32 type in OAS3).

• https://spec.openapis.org/oas/v3.0.3#data-types
• https://www.arin.net/resources/guide/asn/#:~:text=A%204%2Dbyte%20ASN%20is,to

What problem are you facing?

While retrieving connections for a project, using endpoint https://api.equinix.com/metal/v1/projects/{project_id}/connections and below sdk, getting json unmarshalling error for speed parameter for the connection

apiClient.InterconnectionsApi.
	ProjectListInterconnections(context.Background(), project.GetId()).
	Execute()

Note: Any available value of the speed (ex: 5000000 etc) is provided, the error remains the same

Error:

json: cannot unmarshal number 10000000000 into Go struct field _InterconnectionList.interconnections of type int32

How could the Equinix Metal Go SDK help solve this problem?

There seems to be a mismatch in the schema validation on the server and published api doc. Need to fix the marshalling of speed parameter from the server

The Metal API spec does not specify a backend_transfer_enabled property for the Project component, which means we can't read it in the terraform provider: https://github.com/equinix/terraform-provider-equinix/pull/517/files#diff-06c1ecab9b2a05b42b2d8693260c74b084d6cdd8902c905af5c3c8963cd1cac5R150

It is, however, included in the spec for ProjectUpdateInput, so we can change the value: https://github.com/equinix/equinix-sdk-go/blob/main/services/metalv1/model_project_update_input.go#L23C2-L23C24

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

This repository currently has no open or pending branches.

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

When using client-side filtering against a paginated resource list, the pagination helpers could accept criteria functions to determine if pagination can be concluded early.

If there are 1000 devices that I need to sift through to find the one with a particular set of tags (kubernetes-sigs/cluster-api-provider-packet#630), I should be able to take advantage of pagination helpers and not request 100 pages of data if my single desired matching result was found on page 1.

See related packngo issue packethost/packngo#206

The fabric-go GitHub Actions workflows were updated to allow the sync workflow to automatically discover the latest version of the Fabric v4 OpenAPI spec, download that version, and then regenerate code from the new spec. This reduces manual effort to keep the SDK aligned with API changes.

This is the PR that updated the fabric-go automation to pull in the latest Fabric v4 spec instead of requiring a manual PR to bump the spec version number: equinix-labs/fabric-go#30

It would be nice to replicate that work here so that we can avoid reintroducing the requirement for a manual spec version bump and so that we can build some momentum around using the Swagger API registry URL to discover new Equinix service versions (and possibly enable discovery of the services themselves).

Equinix Metal deprecated facilities in favor of metros in April 2023. As of January 2024, facilities have been (largely?) removed from Equinix Metal; to avoid confusion and simplify code, we should remove the deprecated facility-related properties from this SDK entirely.

In terraform-provider-equinix, there's a place where we use the href of a project's organization to get the organization ID without having to include the full organization, which would increase response latency.

The code to do that is more verbose than necessary because the Organization schema does not declare an href property, so we have to read that property from AdditionalProperties and then, because AdditionalProperties can contain properties of any type, we have to cast the href property to a string.

If the Organization schema in the Metal API spec were updated to declare an href property that is type: string, we could simply do organization.GetHref().