Skip to content

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

Merged
riw777 merged 2 commits intodev/10.0from
mergify/bp/dev/10.0/pr-15377
Feb 27, 2024
Merged

lib: Do not convert EVPN prefixes into IPv4/IPv6 if not needed (backport #15377)#15412
riw777 merged 2 commits intodev/10.0from
mergify/bp/dev/10.0/pr-15377

Conversation

@mergify
Copy link

@mergify mergify bot commented Feb 23, 2024

This is an automatic backport of pull request #15377 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

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 <[email protected]>
(cherry picked from commit 439b739)
Test prefix-list matching and some other EVPN stuff.

Signed-off-by: Donatas Abraitis <[email protected]>
(cherry picked from commit 049a2c0)
Copy link
Member

@riw777 riw777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@riw777 riw777 merged commit 7f7325a into dev/10.0 Feb 27, 2024
@mergify mergify bot deleted the mergify/bp/dev/10.0/pr-15377 branch February 27, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev/10.0 libfrr size/M tests Topotests, make check, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants