-
Notifications
You must be signed in to change notification settings - Fork 82
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
Support Linode VPC #362
Support Linode VPC #362
Conversation
61fd5ac
to
83d4815
Compare
83d4815
to
371cd0c
Compare
371cd0c
to
352e258
Compare
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.
Seems to work well in local testing! Could the corresponding events be included in this PR, as well as the firewall_id
field on the Instance POST request?
@lgarber-akamai Thanks for the feedback! How does it look now? |
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.
Great work! Just some small suggestions.
Co-authored-by: Ye Chen <[email protected]>
Description
This PR contains implementations for all Linode VPC related APIs. Tests are failing because we can't put fixtures from production server in the repo.
You can pull and do the integration tests against Alpha APIs with commands below.
This will be merged into a feature branch rather than the main branch, and we can merge to main when the API is in production and we can get fixtures from it.
Test
make fixtures ARGS="-run TestVPC"
make fixtures ARGS="-run TestInstance_Config"