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

VPC IP address GET updates #923

Merged
merged 9 commits into from
Apr 17, 2024

Conversation

Vernholio
Copy link
Collaborator

@Vernholio Vernholio commented Feb 23, 2024

With this update:

  • Added new endpoint: /vpcs/ips (VPC IP Addresses List). Returns a list of all VPC IP addresses on the account.
  • Added new endpoint: /vpcs/:id/ips (IP Address View for a specific VPC). Returns all IP addresses for a specific VPC (:id).
  • Added the "vpc" array to the /linode/instances/{linodeId}/ips (Networking Information List) endpoint. Includes all VPC IP addresses for a specified Linode.
  • Corrected support for nat_1_1 member within the ipv4 object when creating a new Linode.
  • Removed message for IPv6 beta support with VPC's when creating a Linode. (IPv6 is not supported.)

@Vernholio
Copy link
Collaborator Author

Vernholio commented Mar 14, 2024

Originally, I thought an endpoint had been added that mimics the IP Address View operation, and you could target a VPC based on a specific IP address (/networking/vpcs/ips/{address}). I realized that this isn't the case. We've included a derivation of the new endpoint, /networking/vpcs/ips. You can add the id value in the path to target a specific VPC interface to view its specific IP addresses (/networking/vpcs/{id}/ips). I updated this with commit 43c5702.

openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
Copy link

@mette-berger mette-berger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks nice! (Pre-approving as there was just the single small issue).

openapi.yaml Outdated Show resolved Hide resolved
openapi.yaml Outdated Show resolved Hide resolved
Copy link
Member

@zliang-akamai zliang-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

subnet_id seems to be missing?

@Vernholio Vernholio merged commit 3671092 into linode:development Apr 17, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants