Skip to content

feat(frrcfgd): update for bgp locator config#21899

Merged
kperumalbfn merged 1 commit intosonic-net:masterfrom
LARLSN:lsn-frrcfgd-bgp-locator
Mar 13, 2025
Merged

feat(frrcfgd): update for bgp locator config#21899
kperumalbfn merged 1 commit intosonic-net:masterfrom
LARLSN:lsn-frrcfgd-bgp-locator

Conversation

@LARLSN
Copy link
Contributor

@LARLSN LARLSN commented Mar 3, 2025

Why I did it

since srv6 locator has changed as https://github.com/BYGX-wcr/SONiC/blob/srv6_static_config_hld/doc/srv6/srv6_static_config_hld.md,
add support for bgp srv6 locator config load.
Srv6 test config updated in: sonic-net/sonic-mgmt#17283

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Tested on daily jenkins job

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: linsongnan <linsongnan.lsn@alibaba-inc.com>
@LARLSN LARLSN requested a review from lguohan as a code owner March 3, 2025 03:29
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@eddieruan-alibaba
Copy link
Collaborator

Test config changes is at sonic-net/sonic-mgmt#17283

Copy link

@ahsalam ahsalam left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@cscarpitta cscarpitta left a comment

Choose a reason for hiding this comment

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

LGTM

@kperumalbfn
Copy link
Contributor

LGTM

@kperumalbfn kperumalbfn merged commit ed82e02 into sonic-net:master Mar 13, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants