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

Fix DHCP release bug #351

Merged
merged 1 commit into from
Apr 21, 2022
Merged

Fix DHCP release bug #351

merged 1 commit into from
Apr 21, 2022

Conversation

yaocw2020
Copy link
Contributor

@yaocw2020 yaocw2020 commented Jan 18, 2022

  • The macvlan interface should not be deleted until the DHCP client has
    released the IP.

  • If the network manager in the operating system uses DHCP to obtain an IP
    for the network card, it will occupy the DHCP client port. In this case,
    the release function will fail because the unicast connection with the UDP
    socket needs the same port. The solution depends on the pr Change unicast connection of DHCPv4 client as a raw socket insomniacslk/dhcp#458

Related issue: harvester/harvester#1835

Signed-off-by: yaocw2020 [email protected]

@yaocw2020 yaocw2020 force-pushed the release-bug branch 3 times, most recently from 3129d87 to 4158d20 Compare January 18, 2022 16:43
@yaocw2020 yaocw2020 changed the title Fix DHCP release bug [WIP] Fix DHCP release bug Jan 21, 2022
@thebsdbox
Copy link
Collaborator

Awesome, waiting for the dependant PR at this time.

The macvlan interface could not be deleted until the DHCP client has
released the IP.

If the network manager in the operating system uses DHCP to obtain an IP
for the network card, it will occupy the DHCP client port. In this case
the release function will fail because the unicast connection with UDP
socket need the same port. Update dhcp package and use a raw socket.

Signed-off-by: yaocw2020 <[email protected]>
@yaocw2020
Copy link
Contributor Author

We replace the github.com/insomniacslk/dhcp with github.com/harvester/dhcp. Once the DHCP Pr is merged with the upstream repo, we'll take the origin package back.

@yaocw2020 yaocw2020 changed the title [WIP] Fix DHCP release bug Fix DHCP release bug Apr 21, 2022
@thebsdbox thebsdbox merged commit d468340 into kube-vip:main Apr 21, 2022
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