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

Downgrade to frr-7.5.1 #200

Merged
merged 7 commits into from
Aug 18, 2023
Merged

Downgrade to frr-7.5.1 #200

merged 7 commits into from
Aug 18, 2023

Conversation

majst01
Copy link
Contributor

@majst01 majst01 commented Aug 3, 2023

This tries to solve #186 and FRRouting/frr#10298

TODO:

  • try to reproduce with frr-9.0 as well

The Issue can be reproduced with the following steps:

On one of the connected leaves take the switch port where the firewall is connected to down like so:

From a machine in the cluster try to reach all of the machines in the storage network, for example with curl https://

# f4f8b200-deef-11e9-8000-3cecef22f910 is the machineid of the firewall
net show lldp | grep f4f8b200-deef-11e9-8000-3cecef22f910
swp7s1     25G    Default    f4f8b200-deef-11e9-8000-3cecef22f910  lan0
# take this switchport down
sudo ip link set dev swp7s1 down

# Wait a while until frr reached steady state again
# take switch up again
sudo ip link set dev swp7s1 up

Try again to reach all storage servers

@majst01 majst01 requested a review from a team as a code owner August 3, 2023 09:38
@Gerrit91 Gerrit91 merged commit 4deb2a4 into master Aug 18, 2023
@Gerrit91 Gerrit91 deleted the frr-7.5.1-for-firewalls branch August 18, 2023 11:58
@Gerrit91
Copy link
Contributor

Gerrit91 commented Sep 15, 2023

On SONiC, also with FRR 7.5.1, after a leaf going offline the firewall will loose internet connection until FRR is restarted.

Reproducer:

  • Have a firewall in the same rack as the leaf
  • Shutdown spine uplink on a single leaf switch

@mwindower

@majst01
Copy link
Contributor Author

majst01 commented Dec 12, 2023

Tested with frr 9.1 without any downtime

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.

3 participants