Skip to content

Conversation

@zliang-akamai
Copy link
Member

📝 Description

Add Linode Interface data source

✔️ How to Test

TF_ACC=1 go test --tags="integration" -v ./linode/linodeinterface -run TestAccDataSourceLinodeInterface_basic

@zliang-akamai zliang-akamai requested a review from a team as a code owner November 4, 2025 07:14
@zliang-akamai zliang-akamai requested review from Copilot, jriddle-linode and lgarber-akamai and removed request for a team November 4, 2025 07:14
Copy link
Contributor

Copilot AI left a 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.

Copy link
Contributor

@lgarber-akamai lgarber-akamai left a 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!

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.

2 participants