-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[Dual-ToR] The tunnel route of the standby ToR cannot be restored after config reload when the mux ports are admin DOWN #16085
Comments
@lolyu can you help assess if this is an issue you addressed lately? |
Hi @ayurkiv-nvda, this is the same as #11924, the issue is due to the fixes are not included in 202211 branch. |
I can confirm that we had a similar issue for Active-Standby on 202205, and it is fixed now. |
Thanks @ayurkiv-nvda, I could reproduce this on If all ports under a Vlan is admin |
Hello @lolyu |
When all the member port is down, the vlan route is removed from the ASIC, the downstream traffic could not be trapped to the kernel. So the kernel will never try to learn the neighbor -> no FAILED neighbor -> no tunnel route. Will discuss further with the team if this could be further improved. |
Description
Found during running community test dualtor_io/test_link_failure.py::test_active_link_admin_down_config_reload_downstream[active-active]
Steps to reproduce the issue:
config save -y
config reload -y
Describe the results you received:
Routes are not created, traffic will be dropped.
after step#2(shutwown all mux interface) we can see that tunnel route successfully created for both mux server IP and soc IP:
But after step#5
config reload
, there are no routes for mux IPs (192.168.0.2, 192.168.0.4, 192.168.0.6, ... 192.168.0.48):Describe the results you expected:
All needed routes (192.168.0.2, 192.168.0.4, 192.168.0.6, ... 192.168.0.48) are expected to be created after
config reload
even if all ports were in admin down state.Output of
show version
:Output of
show techsupport
:Additional information you deem important (e.g. issue happens only occasionally):
The text was updated successfully, but these errors were encountered: