Skip to content
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

Pass ipv6 address information to Linux GCS #1552

Merged
merged 2 commits into from
Oct 28, 2022
Merged

Conversation

helsaawy
Copy link
Contributor

Currently, HCN endpoint IPv6 settings are not passed into the Linux guest, so containers will not be assigned the v6 addresses that the host endpoint has.

This PR updates the shim to forward IPv6 information to the guest, updates the guest to assign both v4 and v6 addresses, if they are present, and adds tests for ipv6 functionality.

Signed-off-by: Hamza El-Saawy hamzaelsaawy@microsoft.com

Currently, HCN endpoint ipv6 settings are not passed into the Linux
guest, so containers will not be assigned their ipv6 addresses.

Update shim to forward that information into the guest, add updates
the guest to assign both sets of addresses.
Added tests for ipv6 functionality.

Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>
@helsaawy helsaawy requested a review from a team as a code owner October 25, 2022 19:16
@katiewasnothere
Copy link
Contributor

Few comments, otherwise it looks good to me.

internal/guest/network/netns.go Outdated Show resolved Hide resolved
Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>
@helsaawy helsaawy merged commit febe69b into microsoft:main Oct 28, 2022
@helsaawy helsaawy deleted the lcow-ipv6 branch October 28, 2022 23:30
anmaxvl pushed a commit that referenced this pull request Feb 7, 2023
PR is for updating hcsshim ADO branch from github equivalent.

Related work items: #1538, #1539, #1544, #1545, #1550, #1551, #1552, #1553, #1559, #1561
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
* Pass ipv6 address information to Linux GCS

Currently, HCN endpoint ipv6 settings are not passed into the Linux
guest, so containers will not be assigned their ipv6 addresses.

Update shim to forward that information into the guest, add updates
the guest to assign both sets of addresses.
Added tests for ipv6 functionality.

Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>

* PR: vestigial comment, logging

Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>

Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>
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