Skip to content

Add nodebalancers field to VPC Subnet structs#885

Merged
mgwoj merged 3 commits intolinode:mainfrom
mgwoj:feature/TPT-4152-linodego-add-nodebalancers-field-to-vpc-subnet-structs
Feb 4, 2026
Merged

Add nodebalancers field to VPC Subnet structs#885
mgwoj merged 3 commits intolinode:mainfrom
mgwoj:feature/TPT-4152-linodego-add-nodebalancers-field-to-vpc-subnet-structs

Conversation

@mgwoj
Copy link
Contributor

@mgwoj mgwoj commented Feb 3, 2026

📝 Description

It adds nodebalancers field and nested struct to the VPC Subnet struct

✔️ How to Test

To run all unit and integration tests:

make test-unit

make test-int

To run only modified unit and added integration tests:

make TEST_ARGS="-run TestVPCSubnet_Get" test-unit
make TEST_ARGS="-run TestVPCSubnets_List" test-unit

make TEST_ARGS="-run TestVPC_Subnet_WithNodeBalancer" test-int

@mgwoj mgwoj requested a review from a team as a code owner February 3, 2026 11:34
@mgwoj mgwoj requested review from dawiddzhafarov and jriddle-linode and removed request for a team February 3, 2026 11:34
@mgwoj mgwoj force-pushed the feature/TPT-4152-linodego-add-nodebalancers-field-to-vpc-subnet-structs branch 2 times, most recently from 1b61561 to b79eb23 Compare February 3, 2026 12:26
@mgwoj mgwoj force-pushed the feature/TPT-4152-linodego-add-nodebalancers-field-to-vpc-subnet-structs branch from b79eb23 to 9c03300 Compare February 3, 2026 12:28
Copy link
Member

@zliang-akamai zliang-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!

Copy link
Contributor

@ezilber-akamai ezilber-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

@mgwoj mgwoj merged commit f69a89f into linode:main Feb 4, 2026
10 checks passed
@mgwoj mgwoj deleted the feature/TPT-4152-linodego-add-nodebalancers-field-to-vpc-subnet-structs branch February 4, 2026 15:05
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.

3 participants