]> git.puffer.fish Git - matthieu/frr.git/commit
isisd: 'tiebreaker' command line funtionality is inconsistent with its implementation
authorb29332 <bao.zhen@h3c.com>
Thu, 15 Aug 2024 08:20:19 +0000 (16:20 +0800)
committerriw777 <355715+riw777@users.noreply.github.com>
Tue, 4 Feb 2025 11:33:02 +0000 (11:33 +0000)
commit27413dd24a431ac1a67d3e790c3d0e67000fb47b
tree76f8b68035341fd7a7798be388c5ebddff088fce
parent817c2c98230f3825df65283fc68f1b8f8a317d04
isisd: 'tiebreaker' command line funtionality is inconsistent with its implementation

The command fast-reroute lfa tiebreaker [downstream | lowest-backup-metric | node-protecting] index (1-255) [level-1 | level-2] will overwrite configurations with the same index but different types. This is because the index is set as the key in frr-isisd.yang. However, the lfa_tiebreaker_cmp function uses a tuple (index, type) as the key. Therefore, the yang file should be modified to stay in sync with the business logic.

Test Scenario:
On RouterA, first configure fast-reroute lfa tiebreaker downstream index 100 level-1, then configure fast-reroute lfa tiebreaker lowest-backup-metric index 100 level-1, and check the configuration:

!
router isis 10
 fast-reroute lfa tiebreaker lowest-backup-metric index 100 level-1
exit
!

Signed-off-by: baozhen-H3C <bao.zhen@h3c.com>
isisd/isis_cli.c
yang/frr-isisd.yang