matthieu/frr
1
0
Fork 0
forked from Mirror/frr
Code Pull requests Activity

yang: Limit eigrp to just 1 instance per vrf

Browse source 
Currently EIGRP has built in yang code that expects only
1 ASN used per vrf.  Let's just limit the operator from
putting themselves in a bad position by allowing something like
this:

router eigrp 33
....
!
router eigrp 99
...
!

no router eigrp 99 would crash because of assumptions
made in the yang processing.

Let's just hard code that assumption into the EIGRP yang
at the moment such that it will not allow you to enter
a `router eigrp 99` instance at all.

This is purely a software limitation to prevent the code
from violating it's current assumptions.  I do not see
much need to support this at this point in time so I
fixed the problem this way instead of having to possibly
touch a bunch of code.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
This commit is contained in:
Donald Sharp 2025-03-27 08:51:05 -04:00
parent b8dfbbcca9
commit 749dc0c966
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
yang/frr-eigrpd.yang
View file

@ -112,7 +112,7 @@ module frr-eigrpd {
list instance { list instance {
key "asn vrf"; key "asn vrf";
description "EIGRP autonomous system instance"; description "EIGRP autonomous system instance";
must "count(../instance[vrf =current()/vrf]) = 1";
leaf asn { leaf asn {
type autonomous-system; type autonomous-system;
description "Autonomous System Number"; description "Autonomous System Number";