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.

Sign up for GitHub

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

Avoid resetting the VF if the netns does not exist #313

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Avoid resetting the VF if the netns does not exist #313

wants to merge 2 commits into from

Commits on Nov 28, 2024

  1. Add debug logging to cmdDel 

    Signed-off-by: Andrea Panattoni <[email protected]>
    @zeeke
    zeeke committed Nov 28, 2024
    Configuration menu
    Copy the full SHA
    b811f7a View commit details
    Browse the repository at this point in the history

  2. Avoid resetting the VF if the netns does not exist 

    The VF might have been assigned to another running
Pod if the network namespace is not present. In cases
like this, resetting the VF might break the configuration
of the running Pod.
The above scenario might occur when the IPAM plugin takes a lot of time to complete, and the CmdDel gets called
multiple times.

Check if the namespace is still present before resetting the VF

Signed-off-by: Andrea Panattoni <[email protected]>
    @zeeke
    zeeke committed Nov 28, 2024
    Configuration menu
    Copy the full SHA
    ad3c025 View commit details
    Browse the repository at this point in the history