sonic-swss
sonic-swss copied to clipboard
[fpmsyncd] Fpmsyncd Next Hop Table Enhancement
What I did Implementing code changes for https://github.com/sonic-net/SONiC/pull/1425
Why I did it add nexthop group feature to fpmsyncd.
How I verified it
enable/disable nexthop group feature
- Klish will call REST API to configure
feature next-hop-group enable. FEATURE|nexthop_groupwill be created inCONFIG_DB- template
zebra.conf.j2will generatezebra.confwithfpm use-next-hop-groupsifFEATURE|nexthop_groupexists inCONFIG_DB. Else, it will generatezebra.confwithno fpm use-next-hop-groups(default behavior)
- template
- Do
config savecomman and write to/etc/sonic/config_db.json - restart SONiC:
virsh reboot sonic-nhg /etc/frr/zebra.confhasfpm use-next-hop-groupsinstead ofno fpm use-next-hop-groups
Klish CLI for feature nexthop_group
- Enable:
sonic(config)# feature next-hop-group enable - Disable:
sonic(config)# no feature next-hop-group
Enable
admin@sonic:~$ sonic-cli
sonic# configure terminal
sonic(config)#
end Exit to EXEC mode
exit Exit from current mode
feature Configure additional feature
interface Select an interface
ip Global IP configuration subcommands
mclag domain
no To delete / disable commands in config mode
sonic(config)# feature
next-hop-group Next-hop Groups feature
sonic(config)# feature next-hop-group
enable Enable Next-hop Groups feature
sonic(config)# feature next-hop-group enable
<cr>
Disable
sonic(config)# no
feature Disable additional feature
ip Global IP configuration subcommands
mclag domain
sonic(config)# no feature
next-hop-group Disable Next-hop Groups feature
sonic(config)# no feature next-hop-group
<cr>
Details if related
/AzurePipelines run Azure.sonic-swss
Commenter does not have sufficient privileges for PR 2919 in repo sonic-net/sonic-swss
@zice312963205 @shuaishang Please help in restarting the /AzurePipelines run Azure.sonic-swss. Would you be able to grant me the privilege?
@nakano-omw your branch needs to be updated and you need to repush your code first. @prsunny Hi Prince, can you please help with this PR to merge. Thanks.
@ridahanif96 I have repush. Thanks.
Looks good
reviewers, can you please help to review and approve this PR? Thanks.