-
Notifications
You must be signed in to change notification settings - Fork 69
[New] VPC and Configuration Profile Interface endpoints #858
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @bbiggerr,
I have been part of designing and implementing the VPC API changes, but have not worked with the Linode API (or the api-docs) before.
Most of my comments are just suggestions and subtle nuances that we may not want to confuse the user with. I trust that you will know better than me what to include and what to skip. So feel very free to resolve those comments without making any changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the last review I made we have changed some things in the API (earlier on we were not really sure we would be able to get them ready in time for the BETA release).
There are four changes.
- A new property with VPC info has been added to the GET JSON object from the /ntworking/ips and /linode/instances/:id/ips endpoints, see: API spec section on changes to existing endpoints
… Configuration Profile Interface Delete endpoint
Commands Added:
Commands Updated: