-
Notifications
You must be signed in to change notification settings - Fork 81
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
Sure DHCP will work on VLAN interface if VLAN is configured #879
Conversation
Signed-off-by: Jian Wang <[email protected]>
@mergify backport v1.4 v1.3 |
✅ Backports have been created
|
Signed-off-by: Jian Wang <[email protected]>
Another hidden bug: in the code path, the VID is not copid in function
|
Signed-off-by: Jian Wang <[email protected]> Co-authored-by: Kiefer Chang <[email protected]>
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!
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
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, thank you.
Problem:
DHCP looks to have flaws with VLAN interfaces.
Solution:
Sure the DHCP is working on VLAN interface if VLAN is configured.
This PR has 3 commits:
PR #878 is merged to this one.
Co-authored-by: Kiefer Chang [email protected]
Related Issue:
harvester/harvester#6959
harvester/harvester#3428
Test plan:
Per issue description
Local test:
Installer with this fix, the DHCP discovery frame is encaped with VLAN 100 ( and we did not set a DHCP server on VLAN 100)
Original installer without this fix, the DHCP discovery frame is NOT encaped with VLAN 100