-
Notifications
You must be signed in to change notification settings - Fork 109
Implement Linode Interface data source #2159
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
base: dev
Are you sure you want to change the base?
Implement Linode Interface data source #2159
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.
Pull Request Overview
This PR implements a new Linode Interface data source that allows users to query information about Linode network interfaces. The data source supports retrieving details about public, VLAN, and VPC interfaces including their IPv4/IPv6 configurations and routing information.
- Adds comprehensive schema definitions for interface types (public, VLAN, VPC)
- Implements data models and flattening logic for all interface configurations
- Includes test coverage with Terraform configuration templates
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| linode/linodeinterface/tmpl/template.go | Adds template function for generating basic data source test configuration |
| linode/linodeinterface/tmpl/datasource_basic.gotf | Terraform template for testing data source with instance and interface resources |
| linode/linodeinterface/framework_datasource_test.go | Integration test for the data source with basic functionality checks |
| linode/linodeinterface/framework_datasource_schema.go | Schema definitions for all interface types and their nested attributes |
| linode/linodeinterface/framework_datasource_models.go | Data models and flattening logic for converting API responses to Terraform state |
| linode/linodeinterface/framework_datasource.go | Main data source implementation with read logic and API client interaction |
| linode/framework_provider.go | Registers the new data source with the provider |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
LGTM other than the comments posted by Copilot. Nice work!
📝 Description
Add Linode Interface data source
✔️ How to Test