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

Jump to bottom

Fix egress gateway pod cleanup for remote zone pods. #4744

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

Fix egress gateway pod cleanup for remote zone pods. #4744

wants to merge 2 commits into from

Commits on Sep 20, 2024

  1. Add egress gateway e2e test option to delete gateway pod instead of 

    re-labeling gateway pod. It uses different handlers for update and
delete pod internally.
Remove external.gateway from the dual-stack exclusion, as it is
supported for ipv6.

Signed-off-by: Nadia Pinaeva <[email protected]>
    @npinaeva
    npinaeva committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    4b9c42b View commit details
    Browse the repository at this point in the history

  2. Cleanup gateway pod for remote zone. 

    For local zone pods, deleteLogicalPort cleans this up, but before IC
this function was called for all non-host-network pods, hence this
logic. After IC, deleteLogicalPort won't be called for all remote zone
pods, so condition is not needed.

Signed-off-by: Nadia Pinaeva <[email protected]>
    @npinaeva
    npinaeva committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    839842b View commit details
    Browse the repository at this point in the history