-
Notifications
You must be signed in to change notification settings - Fork 78
Project: Linode Interfaces #540
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
* Add support for Firewall Templates * oops * Add LA notices
* Enhanced Interfaces: Add account-related fields * Add setting enum * Add LA notice * Drop residual print
… non-interface networking (#526) * Implement endpoints & fields related to VPCs and non-interface networking * Add LA notices
…enhanced-interfaces
…lds (#533) * Add support for Linode-related endpoints and fields * oops * tiny fixes * fix docsa * Add docs examples * Docs fixes * oops * Remove irrelevant test * Add LA notices * Fill in API documentation URLs * Add return types
…faceOptions firewall ID (#565) * Add ExplicitNullValue support * Fix failing integration tests * Add unit tests * Add docs disclaimer
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.
Pull Request Overview
This PR introduces support for Linode Interfaces, a new network interface management system that provides enhanced control over network configurations for Linode instances. The changes include comprehensive API client support, testing infrastructure, and fixture data to support the new interface generation system.
Key changes:
- Adds new
LinodeInterfaceobjects and related data structures for managing network interfaces - Implements interface settings, upgrade functionality, and firewall integration
- Updates account and region capabilities to include "Linode Interfaces" support
- Expands test coverage with new unit and integration tests
Reviewed Changes
Copilot reviewed 49 out of 49 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| linode_api4/objects/linode_interfaces.py | New file implementing LinodeInterface objects and related data structures |
| linode_api4/objects/linode.py | Adds interface generation support, interface management methods, and upgrade functionality |
| linode_api4/objects/networking.py | Extends networking objects with firewall settings, templates, and interface_id support |
| linode_api4/objects/account.py | Adds interfaces_for_new_linodes setting enum and property |
| linode_api4/groups/networking.py | Adds firewall template/settings support and IPv6 range allocation |
| test/unit/objects/linode_interface_test.py | New comprehensive test file for LinodeInterface functionality |
| Various test fixtures | Updates to support Linode Interfaces capability and new API responses |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
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.
Pull Request Overview
Copilot reviewed 49 out of 49 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
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.
Pull Request Overview
Copilot reviewed 49 out of 49 changed files in this pull request and generated 10 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
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.
Pull Request Overview
Copilot reviewed 49 out of 49 changed files in this pull request and generated 4 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
lgarber-akamai
left a comment
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.
🎉
yec-akamai
left a comment
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 🎉
No description provided.