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

The longest prefix matching table of GK blocks does not accept a default route #91

Closed
AltraMayor opened this issue Apr 17, 2018 · 0 comments
Closed

The longest prefix matching table of GK blocks does not accept a default route #91

AltraMayor opened this issue Apr 17, 2018 · 0 comments
Labels
enhancement
Milestone
Minimal deployments

Comments

@AltraMayor
Copy link
Owner

The longest prefix matching (LPM) table of GK blocks does not accept a default route (i.e. a route for the prefix of length zero) because it uses DPDK's LPM implementation, which doesn't accept a prefix of length zero.

Given that Gatekeeper servers are expected to be BGP speaker, this may not be an issue at all, but someone may find a deployment scenario in which default routes are necessary. Thus, until this deployment scenario becomes known, this issue has very low priority.

A solution is patching DPDK to support zero-length prefixes.
@AltraMayor AltraMayor added this to the Advanced features milestone Apr 17, 2018
@AltraMayor AltraMayor mentioned this issue Apr 17, 2018
Merged
@AltraMayor AltraMayor mentioned this issue Dec 17, 2019
Open
AltraMayor added a commit that referenced this issue Oct 26, 2022
@AltraMayor
gatekeeper: support the zero-length prefix
ad425ed 
The old libraries rte_lpm and rte_lpm6 do not support
the zero-length prefix, but the new libraries rte_fib and
rte_fib6 do.  This commit removes the previous restrictions.

This commit impacts the routing table of Gatekeeper servers, and
the LPM tables used on Lua policies.

This commit closes #91.
AltraMayor added a commit that referenced this issue Nov 1, 2022
@AltraMayor
gatekeeper: support the zero-length prefix
6de1529 
The old libraries rte_lpm and rte_lpm6 do not support
the zero-length prefix, but the new libraries rte_fib and
rte_fib6 do.  This commit removes the previous restrictions.

This commit impacts the routing table of Gatekeeper servers, and
the LPM tables used on Lua policies.

This commit closes #91.
AltraMayor added a commit that referenced this issue Jan 31, 2023
@AltraMayor
gk: adopt the new FIB data structure
edfd653 
This commit replaces the backend libraries rte_lpm and rte_lpm6
with the new FIB library to implement the routing table.

Thanks to the new RIB and FIB libraries, the zero-length prefix
(AKA the default route) is now supported.

This commit closes #91.
AltraMayor added a commit that referenced this issue Feb 2, 2023
@AltraMayor
gk: adopt the new FIB data structure
4a3772e 
This commit replaces the backend libraries rte_lpm and rte_lpm6
with the new FIB library to implement the routing table.

Thanks to the new RIB and FIB libraries, the zero-length prefix
(AKA the default route) is now supported.

This commit closes #91.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
Minimal deployments
Development

No branches or pull requests
1 participant
@AltraMayor