Skip to content

Comments

Adding VPC Enhanced Interfaces#724

Merged
jriddle-linode merged 12 commits intolinode:proj/vpc-linodes-enhanced-interfacesfrom
jriddle-linode:new/interfaces
May 16, 2025
Merged

Adding VPC Enhanced Interfaces#724
jriddle-linode merged 12 commits intolinode:proj/vpc-linodes-enhanced-interfacesfrom
jriddle-linode:new/interfaces

Conversation

@jriddle-linode
Copy link
Collaborator

📝 Description

What does this PR do and why is this change necessary?

Initial additions for interfaces

✔️ How to Test

How do I run the relevant unit/integration tests?

make test-unit

@jriddle-linode jriddle-linode requested a review from a team as a code owner April 17, 2025 21:32
@jriddle-linode jriddle-linode added the new-feature for new features in the changelog. label Apr 17, 2025
@jriddle-linode jriddle-linode requested review from ezilber-akamai and ykim-akamai and removed request for a team April 17, 2025 21:32
@jriddle-linode jriddle-linode changed the base branch from main to proj/vpc-linodes-enhanced-interfaces April 17, 2025 21:32
@ykim-akamai ykim-akamai requested a review from vshanthe April 30, 2025 16:24
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!

@jriddle-linode jriddle-linode merged commit 75f0190 into linode:proj/vpc-linodes-enhanced-interfaces May 16, 2025
10 checks passed
@jriddle-linode jriddle-linode deleted the new/interfaces branch May 16, 2025 19:02
rahulait pushed a commit to rahulait/linodego that referenced this pull request Jun 27, 2025
## 📝 Description

**What does this PR do and why is this change necessary?**

Initial additions for interfaces

## ✔️ How to Test

**How do I run the relevant unit/integration tests?**

```bash
make test-unit
```
rahulait pushed a commit to rahulait/linodego that referenced this pull request Jul 8, 2025
## 📝 Description

**What does this PR do and why is this change necessary?**

Initial additions for interfaces

## ✔️ How to Test

**How do I run the relevant unit/integration tests?**

```bash
make test-unit
```
zliang-akamai added a commit that referenced this pull request Aug 21, 2025
* Implement changes for account APIs (#715)

* Implement changes for account APIs

* Cleanup and re-add tests

* gofumpt

* Add availability notice

* Added support for `networking/` and `vpcs/` endpoint changs for VPC Linodes Enhanced Interfaces (#727)

* Added support for networking/firewalls related changes

* Added support for Networking IPs and VPC changes

* Addressed PR comments

* Add support for firewall template (#726)

* Add firewall template functions

* Add integration tests

* Fix inline doc

* Adding VPC Enhanced Interfaces (#724)

## 📝 Description

**What does this PR do and why is this change necessary?**

Initial additions for interfaces

## ✔️ How to Test

**How do I run the relevant unit/integration tests?**

```bash
make test-unit
```

* Linode instance changes for Linode Interfaces (#723)

* Add linode instance changes for linode interfaces

* Fix interface structs

* Add tests

* go work sum

* Make default firewall IDs pointers (#765)

* Add DoublePointer helper function

* Remove duplicated DoublePointer in test

* Fix

* Add DoublePointerNull

* lint

* Make nullable default firewall IDs pointer

* Make default firewall IDs in update options double pointers

* Update unit test to utilize DoublePointer helper func

* Firewall Related integration tests  (#760)

* firewall_tests

* fix

* fix

* fix_format

* fixPRcomments

* fix

* Linode Interfaces: Consolidate InterfaceGeneration and LinodeInterfaces fields into InstanceCreateOptions (#759)

* Consolidate InterfaceGeneration and LinodeInterfaces fields into InstanceCreateOptions

* Add network_helper to create opts

* Add unmarshal

* Re-run fixtures

* Fix various issues

* Fix firewall settings tests (#772)

* Make interface optional fields pointers (#773)

* Fix firewall settings tests

* Make optional fields pointers for Linode interface structs

* Rename Label field to VLANLabel to align with API attribute naming

* golangci-lint run --fix

* fix a comment

* Nullable default firewall ids (#776)

* Make default_firewall_ids nullable

* Fix unit test

* Add future breaking change notice for ConfigID in VPC IP (#775)

* Add future breaking change notice for ConfigID in VPC IP

* Update wording

* Make FirewallID in linode interface creation options a double ptr (#792)

* Make FirewallID in linode interface creation options a double ptr

* golangci-lint fmt

* Refresh the monthly transfer test fixture

* Make `address` of interface ipv4 optional (#804)

* Make `address` of interface ipv4 optional

* Update test

* Add omitempty

---------

Co-authored-by: Erik Zilber <ezilber@akamai.com>
Co-authored-by: Jacob Riddle <87780794+jriddle-linode@users.noreply.github.com>
Co-authored-by: Vinay <143587840+vshanthe@users.noreply.github.com>
Co-authored-by: Lena Garber <114949949+lgarber-akamai@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-feature for new features in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants