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

lib: Do not convert EVPN prefixes into IPv4/IPv6 if not needed (backport #15377) #15415

Closed
wants to merge 2 commits into from
Closed

lib: Do not convert EVPN prefixes into IPv4/IPv6 if not needed (backport #15377) #15415

wants to merge 2 commits into from

Commits on Feb 23, 2024

  1. lib: Do not convert EVPN prefixes into IPv4/IPv6 if not needed 

    Convert only when this is really needed, e.g. `match ip address prefix-list ...`.

Otherwise, we can't have mixed match clauses, like:

```
match ip address prefix-list p1
match evpn route-type prefix
```

This won't work, because the prefix is already converted, and we can't extract
route type, vni, etc. from the original EVPN prefix.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
(cherry picked from commit 439b739)

# Conflicts:
#	lib/routemap.c
    @ton31337 @mergify
    ton31337 authored and mergify[bot] committed Feb 23, 2024
    Configuration menu
    Copy the full SHA
    e1244fc View commit details
    Browse the repository at this point in the history

  2. tests: Check if multiple route-map match clauses works for EVPN prefixes 

    Test prefix-list matching and some other EVPN stuff.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
(cherry picked from commit 049a2c0)

# Conflicts:
#	tests/topotests/bgp_evpn_route_map_match/r1/frr.conf
#	tests/topotests/bgp_evpn_route_map_match/test_bgp_evpn_route_map_match.py
    @ton31337 @mergify
    ton31337 authored and mergify[bot] committed Feb 23, 2024
    Configuration menu
    Copy the full SHA
    d631963 View commit details
    Browse the repository at this point in the history