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

SelfIP removal can conflict with default route #209

Open
notandy opened this issue Mar 23, 2023 · 1 comment
Open

SelfIP removal can conflict with default route #209

notandy opened this issue Mar 23, 2023 · 1 comment
Assignees
Labels
bug Something isn't working as expected rescheduling Relevant for rescheduling semantics in some way

Comments

@notandy
Copy link
Collaborator

notandy commented Mar 23, 2023

sync_l2_selfips_flow adds and removes selfips if, for example, a LB has been removed from another subnet.
But if this subnet is used as a default GW, this will cause an exception because F5 forbids to remove a selfip in the same subnet as the default GW.

sync_l2_selfips_flow should also update the default route to ensure this error doesn't happens.

@BenjaminLudwigSAP
Copy link
Collaborator

We assume that this didn't lead to bigger issues so far because it is reconciled with the next full sync, which then changes the default route to the remaining subnet, allowing the SelfIP of the now LB-less subnet to be deleted.

@notandy notandy closed this as completed Mar 28, 2023
@notandy notandy reopened this Mar 28, 2023
@BenjaminLudwigSAP BenjaminLudwigSAP self-assigned this Mar 28, 2023
@BenjaminLudwigSAP BenjaminLudwigSAP added the bug Something isn't working as expected label Mar 28, 2023
@BenjaminLudwigSAP BenjaminLudwigSAP added the rescheduling Relevant for rescheduling semantics in some way label Apr 12, 2023
@BenjaminLudwigSAP BenjaminLudwigSAP changed the title sync_l2_selfips_flow removal of selfips can conflict with default route SelfIP removal can conflict with default route Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as expected rescheduling Relevant for rescheduling semantics in some way
Projects
None yet
Development

No branches or pull requests

2 participants