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

Upgrading to 5.2(4) from 4.x can cause /32 static routes to not be installed in FIB #25

Closed
scsheldo opened this issue May 18, 2022 · 2 comments · Fixed by #129
Closed
Assignees

Comments

@scsheldo
Copy link

This seems to be a intended check that the SW does
If it sees /32 static route that also is overlapping with a BD subnet range on the leaf it will not install it into the FIB
RIB yes
The BL with the static route is not affected but other leaf nodes that receive this update from BGP are.
There is no fault thrown as of now and I opened up the issue with devs already
Seems to be related to this https://www.cisco.com/c/en/us/td/docs/switches/datacenter/aci/apic/sw/kb/Cisco-ACI-Floating-L3Out.html#Cisco_Concept.dita_2752bf03-c688-42c0-8721-4d463bd2c4e6
CSCwb91766

@jeestr4d jeestr4d self-assigned this May 7, 2024
@jeestr4d
Copy link
Collaborator

Fix for #25 is https://github.com/jeestr4d/ACI-Pre-Upgrade-Validation-Script/tree/Issue25
local pytest results:

tests/static_route_overlap_check/test_static_route_overlap_check.py::test_logic[icurl_outputs0-4.2(7f)-5.2(8h)-FAIL - OUTAGE WARNING!!] PASSED [ 56%]
tests/static_route_overlap_check/test_static_route_overlap_check.py::test_logic[icurl_outputs1-4.2(7f)-5.2(8h)-FAIL - OUTAGE WARNING!!] PASSED [ 56%]
tests/static_route_overlap_check/test_static_route_overlap_check.py::test_logic[icurl_outputs2-5.0(1h)-6.2(8h)-PASS] PASSED [ 57%]

jeestr4d added a commit that referenced this issue May 22, 2024
@jeestr4d jeestr4d mentioned this issue May 22, 2024
monrog2 added a commit that referenced this issue Aug 12, 2024
* Added logic for CSCwb91766

* Issue #25 Fix

* doc + review

* fix doc

* remove generic try

* finish master merge

* remove comment

---------

Co-authored-by: Gabriel <[email protected]>
@monrog2 monrog2 linked a pull request Aug 12, 2024 that will close this issue
@monrog2
Copy link
Collaborator

monrog2 commented Aug 12, 2024

merged

@monrog2 monrog2 closed this as completed Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants