-
Notifications
You must be signed in to change notification settings - Fork 94
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.
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
[fpmsyncd] Added CLI for Fpmsyncd Next Hop Table Enhancement #122
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Kanji Nakano <[email protected]>
Signed-off-by: Kanji Nakano <[email protected]>
Hi @ayelrod It has been reviewed in Routing WG. Could we merge the PR ? |
<COMMAND name="feature next-hop-group" help="Next-hop Groups feature"></COMMAND> | ||
<COMMAND name="feature next-hop-group enable" help="Enable Next-hop Groups feature"> | ||
<ACTION> | ||
python3 $SONIC_CLI_ROOT/sonic-cli-feature.py configure_sonic_nexthop_groups 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the HLD it is mentioned that the enable disable cannot be changed in run time. Then why do we have a CLI which can modify it? If it doesn't take effect we should have a log message asking the user to save and reboot
Is there an equivalent click-based cli command to enable/disable next-hop group? |
@kwangsuk There is no click-based cli command to enable/disable next hop groups. next-hop group can be enabled/disabled only for KLISH-based cli |
Why I did it
Implementing code changes for sonic-net/SONiC#1425
How to verify it
enable/disable nexthop group feature
feature next-hop-group enable
.FEATURE|nexthop_group
will be created inCONFIG_DB
zebra.conf.j2
will generatezebra.conf
withfpm use-next-hop-groups
ifFEATURE|nexthop_group
exists inCONFIG_DB
. Else, it will generatezebra.conf
withno fpm use-next-hop-groups
(default behavior)config save
comman and write to/etc/sonic/config_db.json
virsh reboot sonic-nhg
/etc/frr/zebra.conf
hasfpm use-next-hop-groups
instead ofno fpm use-next-hop-groups
Klish CLI for feature nexthop_group
sonic(config)# feature next-hop-group enable
sonic(config)# no feature next-hop-group
Enable
Disable